Opened 8 years ago

Closed 8 years ago

#5274 closed Bug (fixed)

[[IE6]] Content Templates dialog is not displaying properly

Reported by: Satya Minnekanti Owned by: Garry Yao
Priority: Normal Milestone: CKEditor 3.3
Component: UI : Dialogs Version: SVN (CKEditor) - OLD
Keywords: IE IBM Confirmed Review+ Cc: joek, Damian

Description

To reproduce the defect

  1. open ajax sample and click on Templates option.
  1. Look at the Content Templates dialog.

In FF the dialog looks proper with Templates Options.

In IE 6 the dialog and the Options are not displayying properly.

I have attached a screenshot to show the differenced between FF & IE6

Attachments (9)

Templates dialog in FF3.jpg (1.0 MB) - added by Satya Minnekanti 8 years ago.
Templates dialog in IE6.jpg (33.9 KB) - added by Satya Minnekanti 8 years ago.
5274.patch (1.5 KB) - added by Garry Yao 8 years ago.
5274_2.patch (1.7 KB) - added by Garry Yao 8 years ago.
5274_3.patch (849 bytes) - added by Garry Yao 8 years ago.
Template Options in IE6 with out BackgroundColor.jpg (28.0 KB) - added by Satya Minnekanti 8 years ago.
Template Options in FF with BackgroundColor.jpg (8.2 KB) - added by Satya Minnekanti 8 years ago.
5274_4.patch (3.2 KB) - added by Garry Yao 8 years ago.
5274_5.patch (3.9 KB) - added by Garry Yao 8 years ago.

Download all attachments as: .zip

Change History (26)

Changed 8 years ago by Satya Minnekanti

Attachment: Templates dialog in FF3.jpg added

Changed 8 years ago by Satya Minnekanti

Attachment: Templates dialog in IE6.jpg added

comment:1 Changed 8 years ago by Satya Minnekanti

Component: GeneralUI : Dialogs
Milestone: CKEditor 3.3

Changed 8 years ago by Garry Yao

Attachment: 5274.patch added

comment:2 Changed 8 years ago by Garry Yao

Keywords: IE Confirmed Review? added
Owner: set to Garry Yao
Status: newassigned
Version: SVN (CKEditor)

Avoid template item size depending on preview image.

comment:3 Changed 8 years ago by Alfonso Martínez de Lizarrondo

Keywords: Review- added; Review? removed

The problem with the patch is that now all the preview images are resized to 100x70. If it's not possible to find another fix for the IE6 problem, I would propose to adjust the changes so this restriction is placed only for IE6 and the rest of browsers can see the images at their correct dimensions.

comment:4 Changed 8 years ago by Garry Yao

Cc: joek Damian Frederico Caldeira Knabben added; joek damo removed
Keywords: Review? added; Review- removed

I assume that all preview images should be chromed as a fixed size?

comment:5 Changed 8 years ago by Frederico Caldeira Knabben

Keywords: Review- added; Review? removed

No, we should not force the image size.

Changed 8 years ago by Garry Yao

Attachment: 5274_2.patch added

comment:6 Changed 8 years ago by Garry Yao

Keywords: Review? added; Review- removed

comment:7 Changed 8 years ago by Frederico Caldeira Knabben

Keywords: Review- added; Review? removed

I just feel that a simpler solution can be found.

Can you confirm that it can be fixed by simply adding onload="this.width=this.width" to the <img> tags? I made some quick tests here and it seems to work. If it works, please provide a new patch with it.

Changed 8 years ago by Garry Yao

Attachment: 5274_3.patch added

comment:8 Changed 8 years ago by Garry Yao

Keywords: Review? added; Review- removed

That's a nice hack.

comment:9 Changed 8 years ago by Frederico Caldeira Knabben

Keywords: Review+ added; Review? removed

Please add a space before "onload".

comment:10 Changed 8 years ago by Garry Yao

Resolution: fixed
Status: assignedclosed

Fixed with [5308].

comment:11 Changed 8 years ago by Satya Minnekanti

Resolution: fixed
Status: closedreopened

Now the Template options are displaying properly in IE6 but the Background colour that surrounds the option when we move through the option is not shown in IE6..it is shown in all other browsers..I am attaching a screen shots to show the differences

Changed 8 years ago by Satya Minnekanti

Changed 8 years ago by Satya Minnekanti

Changed 8 years ago by Garry Yao

Attachment: 5274_4.patch added

comment:12 Changed 8 years ago by Garry Yao

Keywords: Review? added; Review+ removed

comment:13 Changed 8 years ago by Frederico Caldeira Knabben

@satya, please open new tickets instead of reopening tickets that have already been closed, even if they look related.

comment:14 Changed 8 years ago by Frederico Caldeira Knabben

Keywords: Review- added; Review? removed

The background-position trick is really weird (just like IE6), but it works (unlike IE6).

My only concern is regarding the pointer change. I know you wanted to separate the IE6 ugly stuff from the rest, but by doing this we maybe just make the CSS files bigger, having the IE6 impact even more drastic. Couldn't we just change "inherit to hand in the "Fix" declaration that we have a few lines after the patch changes?

Changed 8 years ago by Garry Yao

Attachment: 5274_5.patch added

comment:15 Changed 8 years ago by Garry Yao

Cc: Frederico Caldeira Knabben removed
Keywords: Review? added; Review- removed

I know you wanted to separate the IE6 ugly stuff from the rest...

Actually I wasn't aware of the previous fix at all.

comment:16 Changed 8 years ago by Frederico Caldeira Knabben

Keywords: Review+ added; Review? removed

comment:17 Changed 8 years ago by Garry Yao

Resolution: fixed
Status: reopenedclosed

Fixed with [5393].

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