1162539 - Unable to scroll to top in landscape orientation

Status: RESOLVED FIXED

(Firefox for iOS :: General, defect)

Description

[Aaron Train [:aaronmt]]

Currently there is no status-bar or title-bar visible so one is unable to scroll to top in landscape as opposed to portrait orientation.

Comment 1

[Aaron Train [:aaronmt]]

Safari makes use of a single top of the screen tap to reveal the address bar, and then tapping the top portion of the bar scrolls the page to the top.

Comment 2

[Aaron Train [:aaronmt]]

We should either a) show the navigation bar on top of the screen tap in landscape and or b) designate the top portion as a clickable area to scroll to the top of the page, or c) only do so when the navigation bar is revealed on screen which would require the user scrolling up the page a bit to reveal the navigation bar which is bad

Comment 3

[Wesley Johnston (:wesj)]

UX is going to look into this.

Comment 4

[Robin Andersen [:tecgirl]]

I played with the 'other' browsers a bit to see how they handle this.

Safari's URL bar will reappear if you tap the top of the screen, allowing you to tap a second time to scroll to the top—however, you have to tap in a sweet spot, which is essentially between icons. It's quite difficult to not hit any of the other targets. 

Chrome keeps the status bar in landscape. Easy.

I am for enabling the status bar in landscape. True, we're giving up 40px, but gaining an interaction that would otherwise be difficult to obtain.

Comment 5

[Stephan Leroux [:sleroux]]

Attachment: PR https://github.com/mozilla/firefox-ios/pull/597

I implemented the way Safari does it where you can tap the top area and the toolbars will animate in and if you top on the top area above the address bar field it will scroll you to the top. It doesn't actually feel half bad considering its a small tap target. I also left in the tap to top area when in portrait mode so you can show the url bar easier than scrolling.

Comment 6

[Stefan Arentz | :st3fan | ⏰ EST | he/him]

Comment on attachment 8621597 [details] [review]
PR https://github.com/mozilla/firefox-ios/pull/597

LGTM