Closed Bug 1226655 Opened 9 years ago Closed 9 years ago

With C++ APZ, axis locking behavior is not as good

Categories

(Firefox for Android Graveyard :: Toolbar, defect)

defect
Not set
normal

Tracking

(firefox46 fixed)

RESOLVED FIXED
Firefox 46
Tracking Status
firefox46 --- fixed

People

(Reporter: kats, Assigned: kats)

References

Details

(Whiteboard: [pref-tweak])

Attachments

(1 file, 1 obsolete file)

Using the C++ APZ build for my normal browsing, first thing I noticed was that the axis locking was a lot looser than with the JPZC code. We probably need to just fiddle with the prefs a bit. (in particular the lock angle seems narrower than it used to be)
The lock angle in both cases appears to be 30 degrees, so I'm not sure why it feels different. It might be an implementation difference between the JPZC and the C++ APZ, in which case I don't really want to change it since it will affect B2G as well. I'm getting used to the new behaviour after a day of using it and it's not really "worse", just "different".
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → WONTFIX
We discovered the the axis lock breakout angle is lower than the lock angle by default, so that might explain why the locking is less flaky than it was before. Reopening to play with this.
Status: RESOLVED → REOPENED
Depends on: 1231504
Resolution: WONTFIX → ---
Whiteboard: [pref-tweak]
Attached patch Patch (obsolete) — Splinter Review
This makes things better; now the axis lock angle is actually 30 degrees instead of being artificially restricted to 22.5 degrees because we enter and immediately exit axis lock.
Assignee: nobody → bugmail.mozilla
Attachment #8708442 - Flags: review?(snorp)
Comment on attachment 8708442 [details] [diff] [review] Patch Review of attachment 8708442 [details] [diff] [review]: ----------------------------------------------------------------- OK, I thought maybe that's what was happening.
Attachment #8708442 - Flags: review?(snorp) → review+
This does seem to improve things for me, especially on Amazon's horizontally scrolled areas (which was really annoying before). Anthony, it looks like we didn't allow breakout from the axis lock at all in the old APZ. Do we want to keep that behavior or allow breakout? With this patch the locking seems pretty good, but I don't know if allowing breakout is what we want or not. To be a little more clear, the old behavior was that once you started panning along an axis (vertical or horizontal), we locked it there no matter which angle your touches went once. If you started a vertical pan and then only moved your finger horizontally, we just wouldn't pan at all. The new behavior allows you break out of the lock if you diverge more than 45 degrees from the axis, and it would then presumably also lock to that new axis if you are close enough to it (currently 30 degrees).
Flags: needinfo?(alam)
(In reply to Kartikaya Gupta (email:kats@mozilla.com) from comment #3) > Created attachment 8708442 [details] [diff] [review] > Patch > > This makes things better; now the axis lock angle is actually 30 degrees > instead of being artificially restricted to 22.5 degrees because we enter > and immediately exit axis lock. (In reply to James Willcox (:snorp) (jwillcox@mozilla.com) from comment #5) > This does seem to improve things for me, especially on Amazon's horizontally > scrolled areas (which was really annoying before). > > Anthony, it looks like we didn't allow breakout from the axis lock at all in > the old APZ. Do we want to keep that behavior or allow breakout? With this > patch the locking seems pretty good, but I don't know if allowing breakout > is what we want or not. > > To be a little more clear, the old behavior was that once you started > panning along an axis (vertical or horizontal), we locked it there no matter > which angle your touches went once. If you started a vertical pan and then > only moved your finger horizontally, we just wouldn't pan at all. The new > behavior allows you break out of the lock if you diverge more than 45 > degrees from the axis, and it would then presumably also lock to that new > axis if you are close enough to it (currently 30 degrees). I think we should keep consistent with the previous experience. If I understand correctly, the previous scrolling mechanics didn't allow for ANY sort of breaking? I.e. once it's locked, it's locked. So I don't think we should allow for that right now. Especially since we're making a lot of other changes to our scrolling behaviour. I also just don't have a clear idea for what a good value here would be. So I'm hesitant to prescribe any significant change. But, we can revisit this again after a while if we hear/feel that it needs it. Does that help?
Flags: needinfo?(alam)
Yup, that sounds reasonable. I'll update the patch.
Attached patch Patch v2Splinter Review
Attachment #8708442 - Attachment is obsolete: true
Attachment #8708587 - Flags: review?(snorp)
Attachment #8708587 - Flags: review?(snorp) → review+
Status: REOPENED → RESOLVED
Closed: 9 years ago9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 46
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: