1896029 - "Translation" entry briefly shows in Settings, up every time you open Settings (and then animates closed before you can read it, because it's disabled-by-default)

Status: VERIFIED FIXED

(Firefox for Android :: Translations, defect, P2)

Description

[Daniel Holbert [:dholbert]]

STR:

1. In Firefox Nightly, open the menu and tap Settings
2. Watch the bottom of your screen carefully as the Settings page opens.

ACTUAL RESULTS:
There's a mysteriously-instantly-disappearing section at the bottom of the Settings screen, every time you open it. Looking very closely, you can see that it's labeled "Translation", and it briefly appears between "Language" and "Set as default browser" and then animates away.

EXPECTED RESULTS:
No such immediately-disappearing section. I think this settings subarea is off-by-default, so we should fully hide it by default.

Comment 1

[Daniel Holbert [:dholbert]]

Attachment: screencast of bug

Comment 2

[Daniel Holbert [:dholbert]]

Attachment: cropped slow-motion version of screencast, focusing on the moment where "Translation" is briefly visible

Comment 3

[BugBot [:suhaib / :marco/ :calixte]]

Set release status flags based on info from the regressing bug 1890838

:giorga, since you are the author of the regressor, bug 1890838, could you take a look? Also, could you set the severity field?

For more information, please visit BugBot documentation.

Comment 4

[Iorga Gabriel]

Attachment: Bug 1896029 - "Translation" should be by default invisible in Settings.

Comment 5

[Liz Henry (:lizzard)]

I don't think we need to track this, but it would be nice to verify once it lands in 127.

Comment 6

[Daniel Holbert [:dholbert]]

[Tracking Requested - why for this release]:

(In reply to Liz Henry (:lizzard) (relman/hg->git project) from comment #5)

I don't think we need to track this, but it would be nice to verify once it lands in 127.

Maybe worth tracking to be sure we get the fix uplifted once it lands on nightly which is now 128.

Comment 7

[BugBot [:suhaib / :marco/ :calixte]]

The bug is marked as tracked for firefox127 (beta). However, the bug still has low severity.

:olivia, could you please increase the severity for this tracked bug? If you disagree with the tracking decision, please talk with the release managers.

For more information, please visit BugBot documentation.

Comment 8

[Olivia Hall [:olivia]]

This bug still should be tracked. I moved it up to S3 since it is a very prominent visual glitch and should be uplifted, but it isn't compromising or breaking behavior.

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/01d5edf7604e
"Translation" should be by default invisible in Settings. r=android-reviewers,ohall

Comment 10

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/01d5edf7604e

Comment 11

[Mira Lobontiu (Android QA)]

Verified as fixed on the latest Fenix Nightly 128.0a1 from 5/16, with Samsung Galaxy Tab S8 Ultra (Android 12), and Google Pixel 6 (Android 14).

Comment 12

[BugBot [:suhaib / :marco/ :calixte]]

The patch landed in nightly and beta is affected.
:giorga, is this bug important enough to require an uplift?

• If yes, please nominate the patch for beta approval.
• If no, please set status-firefox127 to wontfix.

For more information, please visit BugBot documentation.

Comment 13

[Iorga Gabriel]

Comment on attachment 9401074 [details]
Bug 1896029 - "Translation" should be by default invisible in Settings.

Beta/Release Uplift Approval Request

• User impact if declined: The user will see a animation when he enters in Setttings.
• Is this code covered by automated tests?: No
• Has the fix been verified in Nightly?: Yes
• Needs manual test from QE?: No
• If yes, steps to reproduce:
• List of other uplifts needed: None
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): Not risky
• String changes made/needed:
• Is Android affected?: Yes

Comment 14

[Pascal Chevrel:pascalc]

Comment on attachment 9401074 [details]
Bug 1896029 - "Translation" should be by default invisible in Settings.

Approved for 127 beta 3, thanks.

Comment 15

[Pulsebot]

https://hg.mozilla.org/releases/mozilla-beta/rev/2caa88a44f57

Comment 16

[Mira Lobontiu (Android QA)]

Verified as fixed on the latest Fenix Beta 127.0b3 with the following devices:

• Google Pixel 6 (Android 14),
• Realme GT Master Edition (Android 13),
• Google Pixel 8 (Android 14), and
• Samsung Galaxy Tab S8 Ultra (Android 12).