]> git.ipfire.org Git - thirdparty/bootstrap.git/commit
Replace dropdown backdrop hack with cleaner JS-only hack
authorPatrick H. Lauke <redux@splintered.co.uk>
Fri, 14 Apr 2017 08:19:00 +0000 (09:19 +0100)
committerGitHub <noreply@github.com>
Fri, 14 Apr 2017 08:19:00 +0000 (09:19 +0100)
commit6d64afe508bfd0bcfb5831a9a4708cef4ad88334
treee42caa655ffa11b5ea8d4dc50340682628a9fd46
parent3b3366e1b6ab30acd8cb468fb6570cca1b38f01e
Replace dropdown backdrop hack with cleaner JS-only hack

* Replace backdrop with simple noop mouse listener

As discussed in https://github.com/twbs/bootstrap/pull/22422 the current
approach of injecting a backdrop (to work around iOS' broken event
delegation for the `click` event) has annoying consequences on
touch-enabled laptop/desktop devices.
Instead of a backdrop `<div>`, here we simply add extra empty/noop
mouse listeners to the immediate children of `<body>` (and remove
them when the dropdown is closed) in order to force iOS to properly
bubble a `click` resulting from a tap (essentially, method 2 from
https://www.quirksmode.org/blog/archives/2014/02/mouse_event_bub.html)
This is sufficient (except in rare cases where the user does manage to tap
on the body itself, rather than any child elements of body - which is not
very likely in an iOS phone/tablet scenario for most layouts) to get iOS to
get a grip and do the correct event bubbling/delegation, meaning the regular
"click" event will bubble back to the `<body>` when tapping outside of the dropdown,
and the dropdown will close properly (just like it already does, even without
this fix, in non-iOS touchscreen devices/browsers, like Chrome/Android and
Windows on a touch laptop).
This approach, though a bit hacky, has no impact on the DOM structure, and
has no unforeseen side effects on touch-enabled laptops/desktops. And crucially,
it works just fine in iOS.

* Remove dropdown backdrop styles

* Update doc for dropdowns and touch-enabled devices
docs/components/dropdowns.md
js/src/dropdown.js
scss/_dropdown.scss
scss/_variables.scss