Opened 16 years ago
Closed 15 years ago
#4092 closed Bug (fixed)
[IE6] Close button missing for dialog without multiple tabs
Reported by: | Garry Yao | Owned by: | Tobiasz Cudnik |
---|---|---|---|
Priority: | Normal | Milestone: | CKEditor 3.0 |
Component: | UI : Dialogs | Version: | |
Keywords: | IE Confirmed Review+ | Cc: |
Description
The close button was not visible when open a dialog which has only one tab.
Reproducing Environment
- IE6/Windows XP SP2
- Standard Mode
- LTR document direction
Attachments (4)
Change History (11)
Changed 16 years ago by
Changed 16 years ago by
Attachment: | 4092.patch added |
---|
comment:3 Changed 16 years ago by
Keywords: | HasPatch added |
---|
The close button's incorrect position in IE6 is due to the following rules which is supposed to be apply to only IE7 quirks only:
/* IE7 + IE quirks */ .cke_skin_kama .cke_browser_ie7.cke_single_page .cke_dialog_contents { float: left; }
It's shown that the dialog style css is still containing multiple classes selectors which is not fixed by #3021. Proposing for changing it, but maybe breaking Tobias' hacks for IE7 RTL at several places, could you please help to verify the patch?
comment:4 Changed 16 years ago by
Owner: | set to Tobiasz Cudnik |
---|---|
Status: | new → assigned |
Changed 16 years ago by
Attachment: | 4092_2.patch added |
---|
comment:5 Changed 16 years ago by
Keywords: | Review? added; HasPatch removed |
---|
Although splitting DOM structure to keep each important class in separate node is highly desired, doing this right now is very dangerous and maybe even time-wasting as dialog layout is going to be re-written.
I'm still unable to reproduce this issue on my IE6 (screenshot attached), but simple patch i'm providing should resolve this issue (as long as quoted css rule is the reason).
comment:6 Changed 16 years ago by
Keywords: | Review+ added; Review? removed |
---|
Screenshot of IE6