Opened 7 years ago

Closed 7 years ago

#6501 closed Bug (fixed)

[IE9] context menu item layout is broken

Reported by: Garry Yao Owned by: Garry Yao
Priority: Must have (possibly next milestone) Milestone: CKEditor 3.4.2
Component: UI : Context Menu Version:
Keywords: IE Cc:

Description

Seen in the attached screenshot, context menu item has smaller occupation as it should.

Attachments (2)

6501.png (12.1 KB) - added by Garry Yao 7 years ago.
context menu layout in IE9
6501.patch (2.9 KB) - added by Garry Yao 7 years ago.

Download all attachments as: .zip

Change History (9)

Changed 7 years ago by Garry Yao

Attachment: 6501.png added

context menu layout in IE9

comment:1 Changed 7 years ago by Garry Yao

Status: newconfirmed

comment:2 Changed 7 years ago by Tobiasz Cudnik

Owner: set to Tobiasz Cudnik
Status: confirmedassigned

comment:3 Changed 7 years ago by Tobiasz Cudnik

Status: assignedpending

Can't reproduce it using Platform Preview 1.9.7766.6000. Unfortunately Beta version doesn't won't to install on my win7, don't know why...

How about someone else would try it on his IE9 ?

comment:4 Changed 7 years ago by Garry Yao

Status: pendingconfirmed

We'll be always talking about IE9 beta instead of Platform Preview for IE9.

Changed 7 years ago by Garry Yao

Attachment: 6501.patch added

comment:5 Changed 7 years ago by Garry Yao

Owner: changed from Tobiasz Cudnik to Garry Yao
Status: confirmedreview

comment:6 Changed 7 years ago by Frederico Caldeira Knabben

Status: reviewreview_passed

I'm just worried about the env.cssClass change, because the IE8 targatted fixes could be potentially useful for IE9 as well. After a quick check, I didn't notice issues, but we may require deeper tests because of this.

comment:7 Changed 7 years ago by Garry Yao

Resolution: fixed
Status: review_passedclosed

Fixed with [5991].

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