Closed Bug 2019120 Opened 3 months ago Closed 3 months ago

Display Blocked by AI Controls Info Message in about:translations

Categories

(Firefox :: Translations, enhancement, P2)

enhancement

Tracking

()

RESOLVED FIXED
150 Branch
Tracking Status
firefox150 --- fixed

People

(Reporter: nordzilla, Assigned: nordzilla)

References

(Blocks 1 open bug)

Details

Attachments

(1 file, 1 obsolete file)

Description

The about:translations design (internal only) needs to display an info message if the Translations feature is blocked by the AI controls.

The info message should also include an "Unblock" button that will unblock only the Translations feature and enable the page.

Priority: -- → P3
Priority: P3 → P2

This commit displays a new info message for when the Translations
feature is blocked by AI Controls. It includes an "Unblock" button
that will unblock the Translations feature for all of Firefox.

This top-level message bar is special in that the main UI remains
visible, but disabled, compared to all of the other top-level message
bars that ensure the main UI is hidden when they are shown.

This commit displays a new info message for when the Translations
feature is blocked by AI Controls. It includes an "Unblock" button
that will unblock the Translations feature for all of Firefox.

This top-level message bar is special in that the main UI remains
visible, but disabled, compared to all of the other top-level message
bars that ensure the main UI is hidden when they are shown.

Attachment #9549237 - Attachment is obsolete: true
Pushed by enordin@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/3aa222d1ad14 https://hg.mozilla.org/integration/autoland/rev/9afe4476df0c Display about:translations Feature Blocked Info Message r=sfoster,fluent-reviewers,bolsson
Pushed by smolnar@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/8f5e1b754a8b https://hg.mozilla.org/integration/autoland/rev/62f142991d00 Revert "Bug 2019120 - Display about:translations Feature Blocked Info Message r=sfoster,fluent-reviewers,bolsson" for causing bc failures @ browser_MLSuggest_integration.js

This is part of a single backout that spanned multiple bugs.

In the patch stack, I increased the timeout when waiting for a mocked RemoteSettings model to download within our Translations tests, hoping that it might also help to reduce intermittent Translations test failures.

There are a few ml related tests that share our Remote Settings mocks. This test case happens to wait for the full duration of the timeout as part of the success path of the test case.

I've reverted the timeout change, and everything should hopefully be fine now.

I would one day like to either fully separate this code, or unify it in a way that ml doesn't rely on Translations under the hood, so that its more clear where the downstream consumers are.

Flags: needinfo?(enordin)
Pushed by enordin@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/5acbdbc4184c https://hg.mozilla.org/integration/autoland/rev/b04a16012a55 Display about:translations Feature Blocked Info Message r=sfoster,fluent-reviewers,bolsson
Status: NEW → RESOLVED
Closed: 3 months ago
Resolution: --- → FIXED
Target Milestone: --- → 150 Branch

Authored by https://github.com/nordzilla
https://github.com/mozilla/enterprise-firefox/commit/3aa222d1ad143f197b5475fbf1a685b356182be8
[enterprise-main] Bug 2019120 - Display about:translations Feature Blocked Info Message r=sfoster,fluent-reviewers,bolsson

Authored by Sandor Molnar
https://github.com/mozilla/enterprise-firefox/commit/8f5e1b754a8b52fdc7a5a2bfd38172e5091051f6
[enterprise-main] Revert "Bug 2019120 - Display about:translations Feature Blocked Info Message r=sfoster,fluent-reviewers,bolsson" for causing bc failures @ browser_MLSuggest_integration.js

Authored by https://github.com/nordzilla
https://github.com/mozilla/enterprise-firefox/commit/5acbdbc4184ca08aef961a4c008e83569c8e68e9
[enterprise-main] Bug 2019120 - Display about:translations Feature Blocked Info Message r=sfoster,fluent-reviewers,bolsson

QA Whiteboard: [qa-triage-done-c151/b150]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: