Ticket #4272 (closed Bug: fixed)

Opened 13 months ago

Last modified 7 months ago

[IE] Kama skin toolbar broken in RTL+quirks

Reported by: tobiasz.cudnik Owned by: m.nguyen
Priority: Normal Milestone: CKEditor 3.2
Component: General Version: SVN (CKEditor) - OLD
Keywords: Confirmed Kama IE Review+ Cc:

Description (last modified by tobiasz.cudnik) (diff)

Kama skin toolbar broken in RTL+quirks. Refer to attached screenshot. Reproduced in IE8.

Attachments

2009-08-21-133252_799x527_scrot.png Download (30.3 KB) - added by tobiasz.cudnik 13 months ago.
config.js Download (0.8 KB) - added by tobiasz.cudnik 13 months ago.
4272.patch Download (1.1 KB) - added by tobiasz.cudnik 12 months ago.
4272_2.patch Download (1.3 KB) - added by m.nguyen 7 months ago.
4272_3.patch Download (1.5 KB) - added by m.nguyen 7 months ago.

Change History

Changed 13 months ago by tobiasz.cudnik

Changed 13 months ago by tobiasz.cudnik

  • keywords IE added
  • description modified (diff)
  • summary changed from Kama skin toolbar broken in RTL+quirks to [IE] Kama skin toolbar broken in RTL+quirks

Changed 13 months ago by tobiasz.cudnik

Changed 12 months ago by tobiasz.cudnik

  • owner set to tobiasz.cudnik
  • status changed from new to assigned

Changed 12 months ago by tobiasz.cudnik

Changed 12 months ago by tobiasz.cudnik

Fix limited to IE quirks only. Tested on IE 6, 7, 8 and seems to work.

Changed 12 months ago by tobiasz.cudnik

  • keywords Review? added

Changed 11 months ago by fredck

  • keywords Review- added; Review? removed

This fix breaks the combo contents positioning. Confirmed in IE8 in LTR after patch.

Changed 9 months ago by fredck

  • milestone changed from CKEditor 3.1 to CKEditor 3.2

Changed 7 months ago by tobiasz.cudnik

  • owner tobiasz.cudnik deleted
  • status changed from assigned to new

Changed 7 months ago by m.nguyen

RTL feature doesn't work in almost browser with version trunk.

Changed 7 months ago by m.nguyen

Changed 7 months ago by m.nguyen

  • keywords Review? added; Review- removed
  • owner set to m.nguyen
  • status changed from new to assigned

Changed 7 months ago by fredck

  • 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).

Changed 7 months ago by m.nguyen

  • keywords Review? added; Review- removed

Changed 7 months ago by fredck

  • keywords Review+ added; Review? removed

Changed 7 months ago by m.nguyen

Changed 7 months ago by m.nguyen

  • status changed from assigned to closed
  • resolution set to fixed

Fixed with [5110].

Changed 7 months ago by m.nguyen

and [5112].

Note: See TracTickets for help on using tickets.