normalized events for ie7/8 for checkboxes and radios fires bound events on propertychange #2352

Open
DimitarChristoff opened this Issue Apr 17, 2012 · 5 comments

Projects

None yet

4 participants

@DimitarChristoff
Member
DimitarChristoff commented Apr 17, 2012 edited

check this fiddle:

http://jsfiddle.net/8VMQs/1/

basically, i'd expect when doing element.set('checked', true); the change event bound NOT to fire automatically.
observed: event fires.

i am trying to find a way to differentiate the event from a UI driven event (eg, one that is a result of a click) but there is nothing in it, it's just propertychange bubbling to change.

relevant bit: https://github.com/mootools/mootools-core/blob/master/Source/Element/Element.Event.js#L170-183

--- Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/55897-normalized-events-for-ie7-8-for-checkboxes-and-radios-fires-bound-events-on-propertychange?utm_campaign=plugin&utm_content=tracker%2F22067&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F22067&utm_medium=issues&utm_source=github).
@arian
Member
arian commented Apr 19, 2012

Also here is another problem described, when you change input.checked in the change event: http://allbtechebooks.blogspot.com/2012/04/mootools-event-causes-infinite-loop-in.html

/cc @csuwldcat

@DimitarChristoff
Member

They ripped off SO - orig here:
http://stackoverflow.com/questions/10182381/mootools-event-causes-infinite-loop-in-ie7-ie8-
its what prmpted me to raise the bug.
On Apr 19, 2012 7:33 PM, "Arian Stolwijk" <
reply@reply.github.com>
wrote:

Also here is another problem described, when you change input.checked in
the change event:
http://allbtechebooks.blogspot.com/2012/04/mootools-event-causes-infinite-loop-in.html

/cc @csuwldcat


Reply to this email directly or view it on GitHub:
#2352 (comment)

@arian
Member
arian commented Apr 19, 2012

Ah, I found it via twitter..

@arian
Member
arian commented Aug 8, 2012

@csuwldcat this is your thing i guess..

@ibolmo ibolmo modified the milestone: 1.5.1, 1.5 Mar 3, 2014
@ibolmo
Member
ibolmo commented Jul 3, 2014

pinging @csuwildcat once more

@SergioCrisostomo SergioCrisostomo modified the milestone: 1.5.2, 1.5.1 Jul 20, 2014
@SergioCrisostomo SergioCrisostomo modified the milestone: 1.5.2, 1.5.3 Sep 14, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment