mozMessageBar.ftl is not pulled into about:translations in some contexts
Categories
(Firefox :: Translations, defect)
Tracking
()
| 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"/>
| Reporter | ||
Comment 1•8 months ago
|
||
Ricardo, are you interested in picking this up as a follow-up to Bug 1815793?
Should be a one-line change.
Comment 2•8 months ago
|
||
Set release status flags based on info from the regressing bug 1815793
| Assignee | ||
Comment 3•8 months ago
|
||
Updated•8 months ago
|
| Assignee | ||
Comment 4•8 months ago
|
||
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.
Comment 6•8 months ago
|
||
| bugherder | ||
Updated•8 months ago
|
Description
•