]> git.ipfire.org Git - thirdparty/bootstrap.git/commit
Fix carousel "hover" behavior on touch-enabled devices
authorPatrick H. Lauke <redux@splintered.co.uk>
Mon, 17 Apr 2017 12:26:46 +0000 (13:26 +0100)
committerGitHub <noreply@github.com>
Mon, 17 Apr 2017 12:26:46 +0000 (13:26 +0100)
commit6ae5fb12e6c996a6526250e8cb703b007e4213fc
tree620408de7fc859370c7abfad37404d0229439e98
parent7ffb61ac5216493bd35a3ff9283f75d58c9ad94f
Fix carousel "hover" behavior on touch-enabled devices

* Add carousel mouse listeners even if touch events enabled

- touch events are enabled not just on "mobile", just also on
touch-enabled desktop/laptop devices; additionally, it's possible to
pair a mouse with traditionally touch-only devices (e.g. Android
phones/tablets); currently, in these situations the carousel WON'T pause
even when using a mouse

* Restart cycle after touchend

as `mouseenter` is fired as part of the touch compatibility events, the
previous change results in carousels which cycle until the user
tapped/interacted with them. after that they stop cycling (as
`mouseleave` is not sent to the carousel after user scrolled/tapped
away).
this fix resets the cycling after `touchend` - essentially returning
to the previous behavior, where on touch the carousel essentially never
pauses, but now with the previous fix it at least pauses correctly for
mouse users on touch-enabled devices.
includes documentation for this new behavior.
docs/components/carousel.md
js/src/carousel.js
js/tests/unit/carousel.js