Opened 10 years ago

Closed 10 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 10 years ago.
context menu layout in IE9
6501.patch (2.9 KB) - added by Garry Yao 10 years ago.

Download all attachments as: .zip

Change History (9)

Changed 10 years ago by Garry Yao

Attachment: 6501.png added

context menu layout in IE9

comment:1 Changed 10 years ago by Garry Yao

Status: newconfirmed

comment:2 Changed 10 years ago by Tobiasz Cudnik

Owner: set to Tobiasz Cudnik
Status: confirmedassigned

comment:3 Changed 10 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 10 years ago by Garry Yao

Status: pendingconfirmed

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

Changed 10 years ago by Garry Yao

Attachment: 6501.patch added

comment:5 Changed 10 years ago by Garry Yao

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

comment:6 Changed 10 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 10 years ago by Garry Yao

Resolution: fixed
Status: review_passedclosed

Fixed with [5991].

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