1847282 - Change the default setting for pull-to-refresh to disabled

Status: VERIFIED FIXED

(Fenix :: Browser Engine, task, P2)

Description

[Jonathan Almeida [:jonalmeida]]

Given the number of bugs we are receiving for pull-to-refresh from users, we should change the default to be disabled.

For users who are not aware of the feature, we have received app reviews and webcompat bugs which affect the product quality.

We do not need to unship this feature, as users that choose to enable it with the buggy behaviour are fine with it.

Comment 1

[eclaudiu64]

And when will all the pull-to-refresh errors be fully resolved?

Comment 2

[Jonathan Almeida [:jonalmeida]]

We're working through the bugs to make the experience better. It's hard to say.

Comment 3

[BMO Github Automation]

Attachment: [mozilla-mobile/firefox-android] Bug 1847282 - Change the default for pull-to-refresh to disabled (#3234)

Comment 4

[BMO Github Automation]

Authored by https://github.com/jonalmeida
https://github.com/mozilla-mobile/firefox-android/commit/ef7ba0668870541a02f7db63d4a689a5f3eb252a
[main] Bug 1847282 - Change the default for pull-to-refresh to disabled

Comment 5

[BMO Github Automation]

Attachment: [mozilla-mobile/firefox-android] Bug 1847282 - Change the default for pull-to-refresh to disabled (backport #3234) (#3236)

Comment 6

[BMO Github Automation]

Attachment: [mozilla-mobile/firefox-android] Bug 1847282 - Change the default for pull-to-refresh to disabled (backport #3234) (#3237)

Comment 7

[BMO Github Automation]

Attachment: [mozilla-mobile/firefox-android] Bug 1847282 - Change the default for pull-to-refresh to disabled (backport #3234) (#3239)

Comment 8

[Jonathan Almeida [:jonalmeida]]

Comment on attachment 9348672 [details] [review]
[mozilla-mobile/firefox-android] Bug 1847282 - Change the default for pull-to-refresh to disabled (backport #3234) (#3237)

This uplift patch was made in error.

Comment 9

[Jonathan Almeida [:jonalmeida]]

Comment on attachment 9348671 [details] [review]
[mozilla-mobile/firefox-android] Bug 1847282 - Change the default for pull-to-refresh to disabled (backport #3234) (#3236)

Beta/Release Uplift Approval Request

• User impact if declined: Users will continue to experience buggy behaviour from pull-to-refresh that we have already identified and are working to resolve.
• Is this code covered by automated tests?: No
• Has the fix been verified in Nightly?: No
• Needs manual test from QE?: Yes
• If yes, steps to reproduce: 1. Download a fresh version of Firefox for Android.

2. Open Settings > Customize > Gestures.
3. Verify that 'Pull to refresh' is disabled by default.

• List of other uplifts needed: None
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): This changes the default value for a pref to disabled which will improve scrolling and gestures.
• String changes made/needed: No
• Is Android affected?: Yes

Comment 10

[Jonathan Almeida [:jonalmeida]]

Comment on attachment 9348674 [details] [review]
[mozilla-mobile/firefox-android] Bug 1847282 - Change the default for pull-to-refresh to disabled (backport #3234) (#3239)

Beta/Release Uplift Approval Request

• User impact if declined: Users will continue to experience buggy behaviour from pull-to-refresh that we have already identified and are working to resolve.
• Is this code covered by automated tests?: No
• Has the fix been verified in Nightly?: No
• Needs manual test from QE?: Yes
• If yes, steps to reproduce: 1. Download a fresh version of Firefox for Android.

2. Open Settings > Customize > Gestures.
3. Verify that 'Pull to refresh' is disabled by default.

• List of other uplifts needed: None
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): This changes the default value for a pref to disabled which will improve scrolling and gestures.
• String changes made/needed: No
• Is Android affected?: Yes

Comment 11

[Jonathan Almeida [:jonalmeida]]

Diana, will this make it in time for the 116 dot release? Apologies in advance, if this causes more work!

Comment 12

[BMO Github Automation]

Authored by https://github.com/mergify[bot]
https://github.com/mozilla-mobile/firefox-android/commit/25fb1d7e7074a5cf7557d4bcbda669ae361f6504
[releases_v117] Bug 1847282 - Change the default for pull-to-refresh to disabled (#3239)

Comment 13

[Dianna Smith [:diannaS]]

Comment on attachment 9348674 [details] [review]
[mozilla-mobile/firefox-android] Bug 1847282 - Change the default for pull-to-refresh to disabled (backport #3234) (#3239)

Approved for Mobile 117.0b7

Comment 14

[Dianna Smith [:diannaS]]

Comment on attachment 9348671 [details] [review]
[mozilla-mobile/firefox-android] Bug 1847282 - Change the default for pull-to-refresh to disabled (backport #3234) (#3236)

Approved for Mobile 116.3.0

Yes it will make it in time. Since it is jut disabling this, ill merge it now.

Comment 15

[BMO Github Automation]

Authored by https://github.com/mergify[bot]
https://github.com/mozilla-mobile/firefox-android/commit/0dbe0595306974e63fcaaeb7f59236dc9481e5e9
[releases_v116] Bug 1847282 - Change the default for pull-to-refresh to disabled (#3236)

Comment 16

[Delia Pop (Android QA)]

Verified as fixed on Beta 117.0b7 and RC 116.3.0, where the "Pull to refresh" toggle is OFF by default.
Tested with:

• Huawei MediaPad M2 (Android 5.1.1)
• Sony Xperia (Android 6.0.1)
• HTC 10 (Android 8)
• Google Pixel 7 (Android 14)

The "Pull to refresh" toggle remains ON by default in Nightly 118.0a1 from 08/14.