diff options
| author |  Andrew Gunnerson <accounts+github@chiller3.com>
| 2023-12-08 10:26:19 -0500 |
|---|---|---|
| committer |  Frédéric Guillot <f@miniflux.net>
| 2024-01-23 19:33:08 -0800 |
| commit | 6648e0af3851f0978cf53f0a7707db1a721d9d7e (patch) | |
| tree | 97779f03d88569d65ea8610c8d3c546b0339afdf /internal/ui/static/js/touch_handler.js | |
| parent | fde84d55ba146a9752e4f20857f3cf063d222cbd (diff) | |
| download | v2-6648e0af3851f0978cf53f0a7707db1a721d9d7e.tar.gz v2-6648e0af3851f0978cf53f0a7707db1a721d9d7e.tar.zst v2-6648e0af3851f0978cf53f0a7707db1a721d9d7e.zip | |
Revert "touch_handler: Fix scroll up behavior on Firefox Android"
This reverts commit 344a237af87e07c51ca73e3b6f1c23598613996d.
The previous behavior is more correct due to the use of preventDefault()
and the commit was introduced only as a workaround. As of [1], the
underlying issue in Firefox has been fixed and downward swipes to scroll
up are no longer ignored every other attempt.
[1] https://bugzilla.mozilla.org/show_bug.cgi?id=1847305
[2] https://bugzilla.mozilla.org/show_bug.cgi?id=1853075
[3] https://bugzilla.mozilla.org/show_bug.cgi?id=1724755
Signed-off-by: Andrew Gunnerson <accounts+github@chiller3.com>
Diffstat (limited to 'internal/ui/static/js/touch_handler.js')
| -rw-r--r-- | internal/ui/static/js/touch_handler.js | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/internal/ui/static/js/touch_handler.js b/internal/ui/static/js/touch_handler.js index a3339cb5..99c1d5b2 100644 --- a/internal/ui/static/js/touch_handler.js +++ b/internal/ui/static/js/touch_handler.js @@ -164,7 +164,7 @@ class TouchHandler { elements.forEach((element) => { element.addEventListener("touchstart", (e) => this.onItemTouchStart(e), hasPassiveOption ? { passive: true } : false); - element.addEventListener("touchmove", (e) => this.onItemTouchMove(e), hasPassiveOption ? { passive: true } : false); + element.addEventListener("touchmove", (e) => this.onItemTouchMove(e), hasPassiveOption ? { passive: false } : false); element.addEventListener("touchend", (e) => this.onItemTouchEnd(e), hasPassiveOption ? { passive: true } : false); element.addEventListener("touchcancel", () => this.reset(), hasPassiveOption ? { passive: true } : false); }); |