Opened 6 years ago

Closed 5 years ago

#8433 closed Bug (fixed)

Revert #6417 to use new events in IE

Reported by: Alfonso Martínez de Lizarrondo Owned by: Alfonso Martínez de Lizarrondo
Priority: Normal Milestone: CKEditor 3.6.3
Component: General Version: 3.4.2
Keywords: Cc:

Description

The way that Microsoft implements some of the new behavior and standards is by using only the W3C event model, leaving the old "MS way" untouched.

That means for example that in order to listen to a file drop we must use addEventListener, if we use attachEvent it won't work.

The fix is just to revert #6417 where the order of setting the listener was changed to fix a problem with IE9 while it was in beta, but the final version doesn't have the bug so I think that we should try to use the W3C way first again

Attachments (1)

8433.patch (1.8 KB) - added by Alfonso Martínez de Lizarrondo 6 years ago.
Proposed patch

Download all attachments as: .zip

Change History (4)

Changed 6 years ago by Alfonso Martínez de Lizarrondo

Attachment: 8433.patch added

Proposed patch

comment:1 Changed 6 years ago by Alfonso Martínez de Lizarrondo

Owner: set to Alfonso Martínez de Lizarrondo
Status: newreview

comment:2 Changed 5 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 3.6.3
Status: reviewreview_passed

comment:3 Changed 5 years ago by Alfonso Martínez de Lizarrondo

Resolution: fixed
Status: review_passedclosed

Fixed with [7381]

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