Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#8349 closed Bug (fixed)

[IE] Editor toolbar outlook meets HTML5 doctype

Reported by: Garry Yao Owned by: Frederico Caldeira Knabben
Priority: Normal Milestone: CKEditor 3.6.2
Component: UI : Toolbar Version:
Keywords: IE HTML5 Cc:

Description

Load any sample page with the HTML5 doctype (<!DOCTYPE html>) under IE9 producing a wrong padding top on the toolbar.

Attachments (1)

8349.patch (497 bytes) - added by Frederico Caldeira Knabben 6 years ago.

Download all attachments as: .zip

Change History (7)

comment:1 Changed 6 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 3.6.2

Changed 6 years ago by Frederico Caldeira Knabben

Attachment: 8349.patch added

comment:2 Changed 6 years ago by Frederico Caldeira Knabben

Owner: set to Frederico Caldeira Knabben
Status: newreview

I'm targeting the fix to all IEs so it'll work for IE 10 as well.

The idea is that we override the original 'display' rule, which is "inline-table". In general, the "inline-table" option has been introduced in IE 8. So, previous versions just see it as a "table" (it's a <table> element, in fact).

All this means that this change should impact on IE 8 and IE 9 only. I've checked both browsers and things seem to be ok. Further testing is needed.

comment:3 Changed 6 years ago by Frederico Caldeira Knabben

#8150 is DUP.

comment:4 Changed 6 years ago by Garry Yao

Status: reviewreview_passed

Looks good.

comment:5 Changed 6 years ago by Frederico Caldeira Knabben

Resolution: fixed
Status: review_passedclosed

Fixed with [7263].

comment:6 Changed 6 years ago by Jakub Ś

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