Fix #13353; #13428; #14359: Better treatment of native-backed events (focus/blur/checkbox-click) #1367
Conversation
|
I have a theory that the people who use the data parameter, or who use a namespace for manually triggering, could just go through the old-style path rather than using a natively triggered event method. That's what they did until I tried to fix it a few versions back. I just can't imagine that fixing these specific cases is worth 484 bytes gzip. |
|
Don't you mean +103? Anyway, you may be right but I don't know how to confirm... it just rubs me the wrong way that we went to so much trouble fixing these events in particular to match native behavior and ended up breaking our own internal consistency. |
|
The size of this fix would likely be reduced by the ideas in http://bugs.jquery.com/ticket/12031#comment:9 as well. |
|
Closing until we get the implementation in http://bugs.jquery.com/ticket/12031 done. |
2 participants
It's big, but much better than gibson042@80789b1... and this brings the count of fixed bugs up to 3, further reducing the added size per fix! 😉
At any rate, I also found some possible optimizations in event.js that I'll be exploring once this PR closes either way.
raw gz Compared to 1.x-master @ fd29642