1136311 - Tweak overscroll prefs

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia, defect)

Description

[Kartikaya Gupta (email:[e-mail])]

Filing this bug for the tweaking of prefs as discussed at https://bugzilla.mozilla.org/show_bug.cgi?id=1066888#c31 and later comments.

Comment 1

[Kartikaya Gupta (email:[e-mail])]

Attachment: Patch

Current proposal by Gordon, waiting feedback from Francis and Jamie.

Comment 2

[Kartikaya Gupta (email:[e-mail])]

https://hg.mozilla.org/integration/mozilla-inbound/rev/323936539fd7

Comment 3

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/323936539fd7

Comment 4

[Kartikaya Gupta (email:[e-mail])]

Comment on attachment 8568705 [details] [diff] [review]
Patch

NOTE: Please see https://wiki.mozilla.org/Release_Management/B2G_Landing to better understand the B2G approval process and landings.

[Approval Request Comment]
Bug caused by (feature/regressing bug #): overscroll implementation
User impact if declined: this patch "loosens" the overscroll a little bit to make it more playful and bouncy, as per UX request. they wanted this change for a while but it was waiting on bug 1042103 to get fixed first.
Testing completed: on master for a couple of days now, haven't heard of anybody complaining. Might want to wait a few more days just in case.
Risk to taking this patch (and alternatives if risky): low-risk pref change
String or UUID changes made by this patch: none

Comment 5

[bhavana bajaj [:bajaj]]

njpark, can you have your eyes on this landing, in-case you see anything funny and also make sure to run your sanity on the build once this lands ?

I am approving this now, but as is really a nice-to-have and not a blocker, we'll backout depending on if we have worse fallouts.

Comment 6

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-b2g37_v2_2/rev/f248015e7f0c

Comment 7

[No-Jun Park [:njpark]]

Checked in on 2.2 today, on almost all types of scroll behavior, and it looks good, and it now stretches as well as the master build. :)