Hide the URL bar if it was shown as a result of navigation

NEW
Unassigned

Status

()

P5
normal
6 years ago
2 months ago

People

(Reporter: Ehsan, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

6 years ago
Currently when you scroll down to hide the URL bar, and click on a link, the URL bar slides down to let you know that things are happening, which is great!  What I would like to see is for it to stay on the screen momentarily when the navigation is done, and then move back up, and let me continue seeing just the content.

Comment 1

6 years ago
This is a UX issue, needinfo'ing ibarlow for comment.
Flags: needinfo?(ibarlow)
As in the title bar would move back up automatically? I'd be open to trying this.
Flags: needinfo?(ibarlow)
(Reporter)

Comment 3

6 years ago
(In reply to comment #2)
> As in the title bar would move back up automatically?

Yes!

Comment 4

6 years ago
Only problem with this is that if you navigated to a new page (which is the likely situation), hiding the toolbar will cause the page to scroll - which the original patch did and we decided we didn't like that...

Do we want to do this anyway?
Flags: needinfo?(ibarlow)
That's the only bit I would be concerned about too. I'd have to try it though to say for sure one way or the other
Flags: needinfo?(ibarlow)
(Reporter)

Comment 6

6 years ago
(In reply to comment #4)
> Only problem with this is that if you navigated to a new page (which is the
> likely situation), hiding the toolbar will cause the page to scroll - which the
> original patch did and we decided we didn't like that...
> 
> Do we want to do this anyway?

I was thinking about loading the page without considering the offset of the toolbar on top of it, and then just slide the toolbar up without touching the page's scroll position.

Comment 7

6 years ago
I don't like the idea of obscuring content for every page load that was initiated while the toolbar wasn't on the screen (which will be most of them).

On the other hand, it's really time I moved onto something else now - after bug 858969, this code is actually quite easy to play with. The margin toolbar position is controlled by the margins on the viewport metrics, and all code that touches this is encapsulated in the LayerMarginsAnimator class.

I'm cc'ing some front-end devs that might want to look into this. Personally, I think there are too many caveats here, and given you're likely to be scrolling content, having the toolbar remain visible after a load is no big deal. (so my vote is wontfix, but this is a ux decision and it sounds like ux want apks to play with).

Comment 8

5 years ago
Removing blocker on bug 716403 - we can ship without this (not to say this shouldn't be explored though)
No longer blocks: 716403

Comment 9

6 months ago
Any progress on this? I agree that it would be nicer behavior to stay hidden.
Re-triaging per https://bugzilla.mozilla.org/show_bug.cgi?id=1473195

Needinfo :susheel if you think this bug should be re-triaged.
Priority: -- → P5
You need to log in before you can comment on or make changes to this bug.