Opened 7 years ago

Closed 6 years ago

#5274 closed Bug (fixed)

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

Reported by: satya Owned by: garry.yao
Priority: Normal Milestone: CKEditor 3.3
Component: UI : Dialogs Version: SVN (CKEditor) - OLD
Keywords: IE IBM Confirmed Review+ Cc: joek, damo

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 7 years ago.
Templates dialog in IE6.jpg (33.9 KB) - added by satya 7 years ago.
5274.patch (1.5 KB) - added by garry.yao 7 years ago.
5274_2.patch (1.7 KB) - added by garry.yao 7 years ago.
5274_3.patch (849 bytes) - added by garry.yao 7 years ago.
Template Options in IE6 with out BackgroundColor.jpg (28.0 KB) - added by satya 7 years ago.
Template Options in FF with BackgroundColor.jpg (8.2 KB) - added by satya 7 years ago.
5274_4.patch (3.2 KB) - added by garry.yao 7 years ago.
5274_5.patch (3.9 KB) - added by garry.yao 7 years ago.

Download all attachments as: .zip

Change History (26)

Changed 7 years ago by satya

Changed 7 years ago by satya

comment:1 Changed 7 years ago by satya

  • Component changed from General to UI : Dialogs
  • Milestone set to CKEditor 3.3

Changed 7 years ago by garry.yao

comment:2 Changed 7 years ago by garry.yao

  • Keywords IE Confirmed Review? added
  • Owner set to garry.yao
  • Status changed from new to assigned
  • Version set to SVN (CKEditor)

Avoid template item size depending on preview image.

comment:3 Changed 7 years ago by alfonsoml

  • 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 7 years ago by garry.yao

  • Cc joek damo fredck 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 7 years ago by fredck

  • Keywords Review- added; Review? removed

No, we should not force the image size.

Changed 7 years ago by garry.yao

comment:6 Changed 7 years ago by garry.yao

  • Keywords Review? added; Review- removed

comment:7 Changed 7 years ago by fredck

  • 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 7 years ago by garry.yao

comment:8 Changed 7 years ago by garry.yao

  • Keywords Review? added; Review- removed

That's a nice hack.

comment:9 Changed 7 years ago by fredck

  • Keywords Review+ added; Review? removed

Please add a space before "onload".

comment:10 Changed 7 years ago by garry.yao

  • Resolution set to fixed
  • Status changed from assigned to closed

Fixed with [5308].

comment:11 Changed 7 years ago by satya

  • Resolution fixed deleted
  • Status changed from closed to reopened

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 7 years ago by garry.yao

comment:12 Changed 7 years ago by garry.yao

  • Keywords Review? added; Review+ removed

comment:13 Changed 7 years ago by fredck

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

comment:14 Changed 7 years ago by fredck

  • 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 7 years ago by garry.yao

comment:15 Changed 7 years ago by garry.yao

  • Cc fredck 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 6 years ago by fredck

  • Keywords Review+ added; Review? removed

comment:17 Changed 6 years ago by garry.yao

  • Resolution set to fixed
  • Status changed from reopened to closed

Fixed with [5393].

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