#4272 closed Bug (fixed)
[IE] Kama skin toolbar broken in RTL+quirks
| Reported by: | Tobiasz Cudnik | Owned by: | Minh Nguyen |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.2 |
| Component: | General | Version: | SVN (CKEditor) - OLD |
| Keywords: | Confirmed Kama IE Review+ | Cc: |
Description (last modified by )
Kama skin toolbar broken in RTL+quirks. Refer to attached screenshot. Reproduced in IE8.
Attachments (5)
Change History (19)
Changed 15 years ago by
| Attachment: | 2009-08-21-133252_799x527_scrot.png added |
|---|
comment:1 Changed 15 years ago by
| Description: | modified (diff) |
|---|---|
| Keywords: | IE added |
| Summary: | Kama skin toolbar broken in RTL+quirks → [IE] Kama skin toolbar broken in RTL+quirks |
Changed 15 years ago by
comment:2 Changed 15 years ago by
| Owner: | set to Tobiasz Cudnik |
|---|---|
| Status: | new → assigned |
Changed 15 years ago by
| Attachment: | 4272.patch added |
|---|
comment:3 Changed 15 years ago by
comment:4 Changed 15 years ago by
| Keywords: | Review? added |
|---|
comment:5 Changed 15 years ago by
| Keywords: | Review- added; Review? removed |
|---|
This fix breaks the combo contents positioning. Confirmed in IE8 in LTR after patch.
comment:6 Changed 14 years ago by
| Milestone: | CKEditor 3.1 → CKEditor 3.2 |
|---|
comment:7 Changed 14 years ago by
| Owner: | Tobiasz Cudnik deleted |
|---|---|
| Status: | assigned → new |
Changed 14 years ago by
| Attachment: | 4272_2.patch added |
|---|
comment:9 Changed 14 years ago by
| Keywords: | Review? added; Review- removed |
|---|---|
| Owner: | set to Minh Nguyen |
| Status: | new → assigned |
comment:10 Changed 14 years ago by
| Keywords: | Review- added; Review? removed |
|---|
- As the name says, "contentsLangDirection" is related to the direction of the contents, not the UI. The UI instead is editor.lang.dir.
- In any case, CSS for the UI must not go inline in the code, but in the skin CSS files. CSS selectors should be used to apply RTL specific style (.cke_rtl).
comment:11 Changed 14 years ago by
| Keywords: | Review? added; Review- removed |
|---|
comment:12 Changed 14 years ago by
| Keywords: | Review+ added; Review? removed |
|---|
Changed 14 years ago by
| Attachment: | 4272_3.patch added |
|---|
Note: See
TracTickets for help on using
tickets.
Fix limited to IE quirks only. Tested on IE 6, 7, 8 and seems to work.