Refine title bar reveal behaviour

RESOLVED FIXED in Firefox 30

Status

()

Firefox for Android
General
RESOLVED FIXED
5 years ago
2 years ago

People

(Reporter: ibarlow, Assigned: lucasr)

Tracking

unspecified
Firefox 30
ARM
Android
Points:
---

Firefox Tracking Flags

(fennec+)

Details

Attachments

(1 attachment)

(Reporter)

Description

5 years ago
We just landed the ability to reveal the title bar by scrolling a short distance, which is great. However, the interaction could still use some refinement, so this is the follow-up bug to take care of that.

Current behaviour: Pull down a certain distance, after which point the title bar can scroll in. However, you have to keep pulling down to reveal the title bar.

Desired behaviour: Pull down a certain distance, but then quickly snap the title bar into place instead of having to pull it in by scrolling further. Our thinking is that this will make the UI feel a little more responsive than it does in the current implementation.

Updated

5 years ago
tracking-fennec: --- → ?
For the record, B2G does the "snapping" of the title bar and I hate it. It's far too jerky and causes the page to feel jumpy while scrolling. I much prefer the current Fennec behaviour.
(Reporter)

Comment 2

5 years ago
I should clarify what I mean by snap, because I agree that it shouldn't act like it does on B2G: the main thing is that it shouldn't push content down like it does on B2G. 

The current interaction we have on Nightly should still remain. The only real change being that once you scroll far enough to 'hook' the title bar and start pulling it down, the title bar should enter the screen at a faster rate than it does now. So while currently it scrolls at a 1:1 rate with page content, perhaps it comes in at a 2:1 rate with page content so it appears in its entirety more quickly.
A few of us also think we should pull down the toolbar sooner. It feels too delayed.
tracking-fennec: ? → +
(Reporter)

Comment 4

5 years ago
Works for me. Let's try both at the same time.
You can tweak this preference to change the panning distance threshold:
http://mxr.mozilla.org/mozilla-central/source/mobile/android/app/mobile.js#438

Using "5" seems to work fine for me.
(Reporter)

Comment 6

5 years ago
5 feels a little quick to me -- we want to account for not only quick retrieval of the bar, but also for the case of scrolling quickly down a page and then backtracking a couple of lines before showing it again. 

I got to a happy place with that at around "10" or so.
(Reporter)

Comment 7

4 years ago
Friendly ping -- forgot about this bug and it would be nice to update that default threshold value.

Updated

4 years ago
OS: Mac OS X → Android
Hardware: x86 → ARM
(Assignee)

Updated

4 years ago
Assignee: nobody → lucasr.at.mozilla
(Assignee)

Comment 8

4 years ago
Created attachment 8375590 [details] [diff] [review]
Reduce scroll threshold to reveal dynamic toolbar to 10% (r=mfinkle)
(Assignee)

Comment 9

4 years ago
Comment on attachment 8375590 [details] [diff] [review]
Reduce scroll threshold to reveal dynamic toolbar to 10% (r=mfinkle)

Use 10% as recommended by ibarlow. I like it.
Attachment #8375590 - Flags: review?(mark.finkle)
Attachment #8375590 - Flags: review?(mark.finkle) → review+
https://hg.mozilla.org/mozilla-central/rev/498de19fecbb
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 30
You need to log in before you can comment on or make changes to this bug.