Don't bubble tech events by default #2351
Conversation
|
Tests passed. Automated cross-browser testing via Sauce Labs and Travis CI shows that the JavaScript changes in this pull request are: CONFIRMED Commit: b14342b (Please note that this is a fully automated comment.) |
|
I thought the last thing we said in chat was to default the whole events system to not bubble by default. |
dmlap
force-pushed
the
simple-events-do-not-bubble
branch
2 times, most recently
from
dac044c to
c8d5dcf
Compare
All events will not bubble unless event.bubbles is strictly true.
|
Tests passed. Automated cross-browser testing via Sauce Labs and Travis CI shows that the JavaScript changes in this pull request are: CONFIRMED Commit: c8d5dcf (Please note that this is a fully automated comment.) |
|
Tests passed. Automated cross-browser testing via Sauce Labs and Travis CI shows that the JavaScript changes in this pull request are: CONFIRMED Commit: dac044c262834e1fb0bd5936f8d2e3d2b45c8414 (Please note that this is a fully automated comment.) |
|
LGTM! Merged. |
The majority of media events are simple events, which mean they don't bubble. Switch the default event behavior to not bubble to match this at the base tech level.