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)
Firefox for Android Graveyard
Toolbar
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)
1.29 KB,
patch
|
snorp
:
review+
|
Details | Diff | Splinter Review |
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)
Assignee | ||
Comment 1•9 years ago
|
||
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
Assignee | ||
Comment 2•9 years ago
|
||
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.
Assignee | ||
Comment 3•9 years ago
|
||
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)
Comment 6•9 years ago
|
||
(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)
Assignee | ||
Comment 7•9 years ago
|
||
Yup, that sounds reasonable. I'll update the patch.
Assignee | ||
Comment 8•9 years ago
|
||
Attachment #8708442 -
Attachment is obsolete: true
Attachment #8708587 -
Flags: review?(snorp)
Attachment #8708587 -
Flags: review?(snorp) → review+
Comment 10•9 years ago
|
||
bugherder |
Status: REOPENED → RESOLVED
Closed: 9 years ago → 9 years ago
status-firefox46:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 46
Updated•4 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•