Opened 6 years ago

Closed 5 years ago

#8433 closed Bug (fixed)

Revert #6417 to use new events in IE

Reported by: alfonsoml Owned by: alfonsoml
Priority: Normal Milestone: CKEditor 3.6.3
Component: General Version: 3.4.2
Keywords: Cc:


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 alfonsoml 6 years ago.
Proposed patch

Download all attachments as: .zip

Change History (4)

Changed 6 years ago by alfonsoml

Proposed patch

comment:1 Changed 6 years ago by alfonsoml

  • Owner set to alfonsoml
  • Status changed from new to review

comment:2 Changed 5 years ago by fredck

  • Milestone set to CKEditor 3.6.3
  • Status changed from review to review_passed

comment:3 Changed 5 years ago by alfonsoml

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

Fixed with [7381]

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