Opened 15 years ago

Closed 15 years ago

#3412 closed Bug (fixed)

Dialog tab bar layout is wrong in IE6.

Reported by: Martin Kou Owned by: Martin Kou
Priority: Normal Milestone: CKEditor 3.0
Component: UI : Dialogs Version: SVN (CKEditor) - OLD
Keywords: Review+ Cc:


To reproduce:

  1. Open replacebyclass.html in IE6.
  2. Open any dialog that has a tab bar (e.g. the image dialog).
  3. The tab bar is seen at a position about 10px lower than the intended location.

The bug occurs in both standards mode and quirks mode. But it doesn't affect IE7.

Attachments (2)

3412.patch (5.8 KB) - added by Martin Kou 15 years ago.
dialog_layout.jpg (51.3 KB) - added by Garry Yao 15 years ago.
Screen shot on Firefox2

Download all attachments as: .zip

Change History (6)

comment:1 Changed 15 years ago by Martin Kou

Keywords: Review? added

IE6 is getting confused by the ".cke_browser_gecko18.cke_skin_v2 ..." CSS declarations added in [3409]. So a bit of restructuring is needed for those CSS classes.

The patch puts the browser related CSS classes one level lower than the DIV containing the skin CSS class. The structural change affects CSS classes defined for other situations such as IE quirks mode and RTL dialogs.

So while testing this patch, be sure to review also:

  1. Quirks mode.
  2. IE6 and IE7.
  3. Firefox 2.
  4. RTL mode.

Changed 15 years ago by Martin Kou

Attachment: 3412.patch added

Changed 15 years ago by Garry Yao

Attachment: dialog_layout.jpg added

Screen shot on Firefox2

comment:2 Changed 15 years ago by Garry Yao

Keywords: Review- added; Review? removed

Broken in FF2 as attached screen shot.

comment:3 Changed 15 years ago by Garry Yao

Keywords: Review+ added; Review- removed

The problem has already been fixed at #3362.

comment:4 Changed 15 years ago by Martin Kou

Resolution: fixed
Status: newclosed

Fixed with [3412].

Click here for more info about our SVN system.

Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy