Opened 7 years ago

Closed 6 years ago

#4459 closed Bug (fixed)

CKEditor maximized appears below select element in IE6

Reported by: wwalc Owned by: garry.yao
Priority: Normal Milestone: CKEditor 3.3
Component: UI : Toolbar Version: 3.1
Keywords: Confirmed Review+ Cc: gogobu@…

Description

To reproduce, add

<select name="foo"><option value="bar">bar</option></select>

above

<label for="editor1">

in replacebycode sample.

In IE6 when CKEditor is maximized, select box appears above CKEditor. It works correctly in FF or IE8 (select box is invisible and only there is only CKEditor on the whole screen).

Attachments (3)

4459.patch (987 bytes) - added by garry.yao 6 years ago.
4459_2.patch (2.6 KB) - added by garry.yao 6 years ago.
4459_3.patch (2.9 KB) - added by garry.yao 6 years ago.

Download all attachments as: .zip

Change History (12)

comment:1 Changed 6 years ago by gogobu

  • Cc gogobu@… added
  • Milestone set to CKEditor 3.2
  • Version set to 3.1

Hope this could be fixed soon as most of our users are still on IE6.

comment:2 Changed 6 years ago by fredck

  • Milestone changed from CKEditor 3.2 to CKEditor 3.3

Changed 6 years ago by garry.yao

comment:3 Changed 6 years ago by garry.yao

  • Component changed from General to UI : Toolbar
  • Keywords Review? added
  • Owner set to garry.yao
  • Status changed from new to assigned

We're free of bug in v2 because of the implicit iframe shim. It's a shame that we still have to work on this in 2010 after IE6's funeral.

comment:4 Changed 6 years ago by alfonsoml

  • Keywords Review- added; Review? removed

Besides <select> also plugins can leak through a div, so the solution is to use a shim iframe (also for the dialog cover).

Taking into account the problem with flash movies, maybe the iframe should be generated for all the browsers?

Changed 6 years ago by garry.yao

comment:5 Changed 6 years ago by garry.yao

  • Keywords Review? added; Review- removed

I'm providing a patch here, but it sometimes crash the browser for me, not sure if it's only happened in my IE6.

comment:6 Changed 6 years ago by alfonsoml

  • Keywords Review- added; Review? removed

Please, don't use CSS expressions as they are known to have a very poor performance. And if it has crashed the browser in a few tests, you can be sure that people that use the editor everyday with IE6 will face the crash, so that must be fixed before requesting review. Having a <select> in the middle of the editor is ugly, having a crash means that all my data has been lost.

Changed 6 years ago by garry.yao

comment:7 Changed 6 years ago by garry.yao

  • Keywords Review? added; Review- removed

You're right but I'm not sure whether the crash problem is caused by CSS expressions, anyway let's try in another more unobtrusive way.

comment:8 Changed 6 years ago by fredck

  • Keywords Review+ added; Review? removed

comment:9 Changed 6 years ago by garry.yao

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

Fixed with [5392].

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