Opened 7 years ago

Closed 7 years ago

#9656 closed Bug (duplicate)

Should event.on/removeListener handle one listener attached multiple times to one object,event?

Reported by: Piotrek Koszuliński Owned by:
Priority: Normal Milestone:
Component: General Version: 4.0
Keywords: Discussion Cc:

Description

Question based on http://dev.ckeditor.com/ticket/9622#comment:4

Solution (or lack of it) should be discussed first.

Change History (2)

comment:1 Changed 7 years ago by Garry Yao

Keywords: Discussion added
Milestone: CKEditor 4.0.1
Status: newconfirmed

It's commonly rejected (checked DOM listener behavior), so let's simply have event::on returns false for the secondary attempt, this will help to prevent further error if one count on the return value, to remove listener, like in #9622.

In any case, it would rarely happen in common sense.

comment:2 Changed 7 years ago by Piotrek Koszuliński

Resolution: duplicate
Status: confirmedclosed

DUP of #9831.

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