705358 - Over scroll / scroll should not exist when page is fully zoomed out and all the content is in the view port.

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect, P3)

Description

[Patryk Adamczyk [:patryk] UX]

Attachment: example of the overscroll I am referring to in the description.

When all the content is displayed in the view port and the display is fully zoomed out (ie. loading about:firefox) horizontal scroll or over scroll should not exist > Reference iOS or Android's native browser. 

Vertical over scroll should be minimized and ideally removed, pull down over scroll has become a more common touch gesture for refreshing content so this would be a nice feature in web apps, but could be useful throughout the browser. 

Over scroll should have a purpose and not just be playful, minimizing the elasticity will create a more solid feel.

Comment 1

[Chris Lord [:cwiiis]]

I agree with this, though we'll need some UX comment.

Generally, I think our default behaviour should match default android before it matches iOS/Samsung tweaks (see bug #705246), but these can be dealt with separately.

Comment 2

[Patryk Adamczyk [:patryk] UX]

This is coming from UX (from me and Madhava).
I should have been clear, yes the behaviour should be the same as the android browser meaning at zoom out (if the entire page's content is in the view port), no over scroll whatever.

I should have been clearer in the initial bug comment.

Comment 3

[Patrick Walton (:pcwalton)]

Agreed that we should match iOS here. I was concerned when the native about:home landed that its native-ness leaks through to the user, since overscroll works in content but doesn't work for the native about:home. I think that this would fix this problem, since about:home would be treated the same as any other page with fully visible content. (about:home does have scrollable "iframe" workalikes, but those shouldn't overscroll anyway.)

Comment 4

[Brad Lassey [:blassey] (use needinfo?)]

I don't understand this bug, either we want over scroll or not. I'm marking this as won't fix. Madhava, if you reopen, please adjust the summary to be clear about what you want.

Comment 5

[Patrick Walton (:pcwalton)]

Attachment: Proposed patch.

We talked about this in the war room today. Overscroll shouldn't happen on axes that are fully zoomed out. No OS, not even iOS, does this.

Here's a patch that implements the behavior.

Comment 6

[Patrick Walton (:pcwalton)]

To be more clear, here's an example use case: If you open a mobile site that has no content off to the left or right of the screen, you shouldn't be able to overscroll to the left or right, because the site is asking for a vertical layout. Similar logic applies to a desktop site that's fully zoomed out to the width of the device.

Think of it as an extra unbreakable axis lock: when there's no reason the user could possibly want to scroll in a direction, we should just lock that direction.

As mentioned, this mirrors iOS's behavior.

Comment 7

[Patryk Adamczyk [:patryk] UX]

Can you let me know when this reaches nightly? Thanks!

Comment 8

[Patrick Walton (:pcwalton)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/c28beca4a896

Comment 9

[u88484]

This is probably the biggest annoyance I have since native UI landed.  It serves no purpose what so ever and make scrolling up and down pages suck badly because the axis lock also doesn't work well so pages scroll in a wavy way.

Comment 10

[Patrick Walton (:pcwalton)]

(In reply to Kurt Schultz (supernova_00) from comment #9)
> This is probably the biggest annoyance I have since native UI landed.  It
> serves no purpose what so ever and make scrolling up and down pages suck
> badly because the axis lock also doesn't work well so pages scroll in a wavy
> way.

This should hit nightly in a day or two -- tree closure has prevented it from making it in before, but it is on its way.

Comment 11

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/c28beca4a896

Comment 12

[Andreea Pod]

Verified fixed on:
-build:  Firefox for Android 22.0a1 (2013-02-27)
-device: Samsung Galaxy Nexus
-OS: Android 4.2.2