1387078 - Disable one-touch-pinch feature in 55 release

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: Toolbar, enhancement)

Description

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

Per the email thread discussion, we should considering disabling the one-touch-pinch feature in 55 release while we polish it a bit more.

Comment 1

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

Attachment: Disable one-touch-pinch

Approval Request Comment
[Feature/Bug causing the regression]: New zoom gesture introduced in bug 1111333
[User impact if declined]: Andreas Bovens reports that in some cases the feature feels buggy/unpolished. It's true that we probably haven't gotten as much user testing of this feature as we would like as it's not a very discoverable gesture and we have low pre-release user populations anyway
[Is this code covered by automated tests?]: not particularly
[Has the fix been verified in Nightly?]: no, requesting disabling the pref on release only for now
[Needs manual test from QE? If yes, steps to reproduce]: yes. try using the gesture with the pref disabled
[List of other uplifts needed for the feature/fix]: none
[Is the change risky?]: not really
[Why is the change risky/not risky?]: just a pref flip. the code it covers is also relatively small and self-contained.
[String changes made/needed]: none

Comment 2

[Sylvestre Ledru [:Sylvestre]]

Comment on attachment 8893430 [details] [diff] [review]
Disable one-touch-pinch

ok, let's take it in rc3

Comment 3

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-release/rev/87438d030cb5

Comment 4

[Jan Henning [:JanH]]

Our Play Store release notes still mention this feature - maybe they should mirror https://www.mozilla.org/en-US/firefox/android/55.0/releasenotes/ and mention bug 1331154 instead?

Comment 5

[Sylvestre Ledru [:Sylvestre]]

Delphine, could you please remove it from the 55 release notes?
I hope it will show immediately here: https://l10n.mozilla-community.org/stores_l10n/product/fx_android/release/whatsnew/raw/
and I will be able to upload it right away

Comment 6

[Delphine Lebédel [:delphine - Use needinfo]]

Thanks Sylvestre. We're traveling to Paraguay now and are trying to get around the bad connection. Flod's on it as he can directly push the changes. Will let you know.

Comment 7

[Francesco Lodolo [:flod]]

Updated. It might seem obvious from the discussion, but it helps to mention the exact words to be removed, I had a hard time matching "one-touch-pinch" with "Double tap and drag gesture allows one-handed zooming, consistent with other Android applications".

Comment 8

[Sylvestre Ledru [:Sylvestre]]

worked great, just a small bug that I reported here: https://github.com/mozilla-l10n/stores_l10n/issues/145

We should not have the string in a few minutes.