Ticket #3414 (closed Bug: fixed)

Opened 6 years ago

Last modified 6 years ago

Combo boxes are incorrectly positioned in IE quirks mode.

Reported by: martinkou Owned by: martinkou
Priority: Normal Milestone: CKEditor 3.0
Component: UI : Floating Panel Version: SVN (CKEditor) - OLD
Keywords: IE Review+ Cc:


To reproduce:

  1. Remove the doctype line from sample.html.
  2. Open replacebyclass.html in IE6.
  3. Open any combo box - e.g. the style combo.
  4. The position of the combo menu is a few pixels off from where it should be.

Other browsers are not affected.


3414.patch (1.2 KB) - added by martinkou 6 years ago.

Change History

comment:1 Changed 6 years ago by martinkou

  • Status changed from new to assigned
  • Owner set to martinkou

Changed 6 years ago by martinkou

comment:2 Changed 6 years ago by martinkou

  • Keywords Review? added

The problem comes from how IE treats the body element in quirks mode - it treats that as the browser window view pane itself. Worse, by "view pane" it includes the IE window border as well.

We're getting this 2px error because IE quirks is reporting the 2px IE windows border as clientTop and clientLeft of the body element. They should be disregarded because we can never draw outside of the window borders anyway. Also, a bit of experiment shows that we can never have a non-zero offsetTop or offsetLeft for body in IE quirks mode because we can't move the browser window's view pane in JavaScript.

So the whole if statement at L1137 - L1141 in element.js should be ignored for IE quirks mode.

comment:3 Changed 6 years ago by fredck

  • Keywords Review+ added; Review? removed

There is no need to cache CKEDITOR.env.ie and CKEDITOR.env.quirks. The first is a special case which is automatically changed to a single char by the packager, while the second is used only once. Please fix it before committing.

comment:4 Changed 6 years ago by martinkou

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

Fixed with [3247].

Click here for more info about our SVN system.

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