Opened 10 years ago

Closed 10 years ago

#3742 closed Bug (fixed)

[IE] Wrong dialog layout for dialogs without tab bar in IE RTL mode

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

Description

Attachments (7)

3742.patch (1003 bytes) - added by Tobiasz Cudnik 10 years ago.
3742_IE_Screenshots.zip (242.6 KB) - added by Frederico Caldeira Knabben 10 years ago.
3742_2.patch (1.2 KB) - added by Tobiasz Cudnik 10 years ago.
3742_3.patch (1.2 KB) - added by Tobiasz Cudnik 10 years ago.
3742_3 Screenshots.zip (473.9 KB) - added by Frederico Caldeira Knabben 10 years ago.
3742_4.patch (2.5 KB) - added by Tobiasz Cudnik 10 years ago.
3742_4.zip (441.1 KB) - added by Tobiasz Cudnik 10 years ago.
Screenshots.

Download all attachments as: .zip

Change History (16)

comment:1 Changed 10 years ago by Tobiasz Cudnik

Keywords: IE Confirmed added
Owner: changed from Martin Kou to Tobiasz Cudnik
Status: newassigned
Summary: Wrong dialog layout for dialogs without tab bar in IE RTL mode[IE] Wrong dialog layout for dialogs without tab bar in IE RTL mode

Changed 10 years ago by Tobiasz Cudnik

Attachment: 3742.patch added

comment:2 Changed 10 years ago by Tobiasz Cudnik

Keywords: Review? added

Changed 10 years ago by Frederico Caldeira Knabben

Attachment: 3742_IE_Screenshots.zip added

comment:3 Changed 10 years ago by Frederico Caldeira Knabben

Keywords: Review- added; Review? removed

Things are still not good with the new patch. I'm attaching screenshots takes with all IE variations that shows the problems.

The screenshots are based on a test page I have locally. It should the editor in LTR and RTL (left and right panes), and Standards and Quirks (top and bottom panes).

Changed 10 years ago by Tobiasz Cudnik

Attachment: 3742_2.patch added

comment:4 Changed 10 years ago by Tobiasz Cudnik

Keywords: Review? added; Review- removed

Patch makes dialog close button working on all IE versions, but rendered dialog aren't pixel-to-pixel identical. In some places they differs in space width.

Changed 10 years ago by Tobiasz Cudnik

Attachment: 3742_3.patch added

comment:5 Changed 10 years ago by Tobiasz Cudnik

Third patch seems to resolve this issue.

comment:6 Changed 10 years ago by Frederico Caldeira Knabben

Keywords: Review- added; Review? removed

Well, if we're supposed to fix it, let's do it the right way, and just once.

I'm attaching new screenshots. Please test it properly before providing a new patch.

Changed 10 years ago by Frederico Caldeira Knabben

Attachment: 3742_3 Screenshots.zip added

Changed 10 years ago by Tobiasz Cudnik

Attachment: 3742_4.patch added

Changed 10 years ago by Tobiasz Cudnik

Attachment: 3742_4.zip added

Screenshots.

comment:7 Changed 10 years ago by Tobiasz Cudnik

Keywords: Review? added; Review- removed

I'm proposing new patch which seems to work well for both close button and tabs space in all IE modes.

I'm attaching screenshots presenting dialogs with and without tabs in all IE modes.

comment:8 Changed 10 years ago by Frederico Caldeira Knabben

Keywords: Review+ added; Review? removed

Please add also the following to dialog.css (line 519) when committing:

.cke_skin_kama .cke_browser_ie6 .cke_dialog_close_button,
.cke_skin_kama .cke_browser_iequirks .cke_dialog_close_button
{
	top: 7px;
}

This will fix the close button position on IE Quirks and IE6.

comment:9 Changed 10 years ago by Tobiasz Cudnik

Resolution: fixed
Status: assignedclosed

Fixed with [3858]. I've included the 2px fix for IE Quirks and IE6.

Note: See TracTickets for help on using tickets.
© 2003 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy