Closed Bug 1960409 Opened 8 months ago Closed 8 months ago

mozMessageBar.ftl is not pulled into about:translations in some contexts

Categories

(Firefox :: Translations, defect)

defect

Tracking

()

RESOLVED FIXED
139 Branch
Tracking Status
firefox-esr128 --- unaffected
firefox137 --- unaffected
firefox138 --- unaffected
firefox139 --- fixed

People

(Reporter: nordzilla, Assigned: ricardo.delgado123)

References

(Regression)

Details

(Keywords: regression)

Attachments

(1 file)

Description

Bug 1815793 updated the about:translations page to use MozMessageBar.

Unfortunately, there are some contexts, such as in the test suite, where the mozMessageBar.ftl is not being pulled in.

This doesn't cause failures in the test, but it does produce an uncaught console error that I did not notice during initial code review.


Steps to Implement

This should be an easy fix by ensuring the .ftl file is present in the HTML:

--- a/toolkit/components/translations/content/translations.html
+++ b/toolkit/components/translations/content/translations.html
@@ -14,6 +14,7 @@
     <link rel="stylesheet" href="chrome://global/skin/in-content/common.css">
     <link rel="stylesheet" href="chrome://global/content/translations/translations.css">
     <link rel="localization" href="toolkit/branding/brandings.ftl"/>
+    <link rel="localization" href="toolkit/global/mozMessageBar.ftl"/>
     <link rel="localization" href="locales-preview/aboutTranslations.ftl"/>

Ricardo, are you interested in picking this up as a follow-up to Bug 1815793?

Should be a one-line change.

Flags: needinfo?(ricardo.delgado123)

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

Assignee: nobody → ricardo.delgado123
Status: NEW → ASSIGNED

Definitely! I added the change to a patch and checked both the live and test versions of about:translations. No more console warnings and the test version now properly displays the message bar text.

Flags: needinfo?(ricardo.delgado123)
Pushed by enordin@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/3a5f2245a82e Add mozMessageBar.ftl localization link to about:translations. r=nordzilla,translations-reviewers
Status: ASSIGNED → RESOLVED
Closed: 8 months ago
Resolution: --- → FIXED
Target Milestone: --- → 139 Branch
QA Whiteboard: [qa-triage-done-c140/b139]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: