Closed Bug 1781209 Opened 2 years ago Closed 2 years ago

Swipe navigation does not work when kinetic scroll is disabled

Categories

(Core :: Widget: Gtk, defect, P3)

Desktop
Linux
defect

Tracking

()

VERIFIED FIXED
105 Branch
Tracking Status
firefox105 --- verified

People

(Reporter: heftig, Assigned: rzvncj)

References

Details

Attachments

(1 file)

I disable kinetic scrolling by turning off apz.gtk.kinetic_scroll.enabled, as without hold gestures, kinetic scrolling irritates me more than it helps.

Unfortunately, this also disables the swipe gestures added in bug 1539730. It would be nice if those were still available.

(Hold gestures are supported by GTK 4, but AFAIK aren't going to be backported to GTK 3.)

Flags: needinfo?(jwatt)
Flags: needinfo?(botond)
Priority: -- → P3

Currently, kinetic scrolling is tied to our "precision" touchpad event handling codepath (touchpad scroll events are turned into PanGestureInput). Setting apz.gtk.kinetic_scroll.enabled=false disables that codepath altogether (touchpad scroll events are turned into ScrollWheelInput). Swipe navigation requires PanGestureInput events.

We could decouple the two, and have a way to take the PanGestureInput codepath without triggering a fling animation at the end of a pan gesture.

We'd probably still want to have a way for users to opt into the ScrollWheelInput codepath, as the scrolling physics are a bit different there and some users might prefer it.

cc Razvan in case this sounds interesting to you

Flags: needinfo?(jwatt)
Flags: needinfo?(botond)
Assignee: nobody → rzvncj
Status: NEW → ASSIGNED
Attachment #9289245 - Attachment description: Bug 1781209 - Swipe navigation does not work when kinetic scroll is disabled. r?dlrobertson,tnikkel → Bug 1781209 - Swipe navigation does not work when kinetic scroll is disabled. r?botond,dlrobertson,tnikkel
Pushed by bballo@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/4378116a135b
Swipe navigation does not work when kinetic scroll is disabled. r=botond
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 105 Branch

Seems to work. Thanks!

Status: RESOLVED → VERIFIED

Happy to help!

You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: