Closed
Bug 1507595
Opened 6 years ago
Closed 6 years ago
Convert about:support to Fluent
Categories
(Core :: Internationalization, defect, P3)
Tracking
()
RESOLVED
FIXED
mozilla66
Tracking | Status | |
---|---|---|
firefox66 | --- | fixed |
People
(Reporter: masterkrombi, Assigned: jaws)
References
Details
Attachments
(3 files, 1 obsolete file)
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/70.0.3538.77 Safari/537.36
Assignee | ||
Updated•6 years ago
|
Reporter | ||
Comment 1•6 years ago
|
||
Comment 2•6 years ago
|
||
Comment on attachment 9025531 [details] Bug 1507595 Convert about:support to Fluent Things I found during porting to TB: + <th data-l10n-id="security-softawre-type"/> Typo, this should be security-software-type. - <h2 class="major-section"> - &aboutSupport.graphicsTitle; - </h2> + <h2 class="major-section" data-l10n-name="graphics-title"/> This should be data-l10n-id. + <th colspan="2" class="title-column" data-l10n-id="graphics-crash-guard-title"/> This should be graphics-crash-guards-title.
Updated•6 years ago
|
Priority: -- → P3
Assignee | ||
Updated•6 years ago
|
Assignee: masterkrombi → jaws
Assignee | ||
Comment 4•6 years ago
|
||
Updated•6 years ago
|
Attachment #9025531 -
Attachment is obsolete: true
Pushed by jwein@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/28805dd1b97a Convert about:support to Fluent. r=Gijs,flod https://hg.mozilla.org/integration/autoland/rev/485ea25a3b82 Use data-l10n-id instead of formatValue where the change is relatively straightforward. r=Gijs
Comment 6•6 years ago
|
||
Backed out 2 changesets (Bug 1507595) for Android build bustages, missing aboutSupport.dtd push that caused the backout: https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception&classifiedState=unclassified&revision=485ea25a3b82fcf3136bd6102313f76e8d955ef4 failure: https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception&classifiedState=unclassified&selectedJob=218667868&searchStr=android%2C4.2%2Cx86%2Copt%2Cbuild-android-x86%2Fopt%2C%28b%29 backout: https://hg.mozilla.org/integration/autoland/rev/8c35900a1c7291259f88caafc8a1f5a9345902d1
Flags: needinfo?(m_kato)
Comment 7•6 years ago
|
||
There are also other failures caused by this bug: https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception&failure_classification_id=2&fromchange=49a28ec78bfef405d2a3dc9214bbd8c15cdf4c9a
Assignee | ||
Updated•6 years ago
|
Flags: needinfo?(m_kato) → needinfo?(jaws)
Assignee | ||
Comment 8•6 years ago
|
||
There are multiple fixes contained in this patch: - fixes for Android/mobile support - adding extra strings to the ftl file that were never defined before, and previously the ID was just printed to the page - update test_l10n.py to not rely on about:support anymore - changing Troubleshoot.jsm to return and ID and args for the strings that should be displayed, so it is compatible with the Fluent API - misc. fixes so strings that are not localized don't go through the Fluent codepath
Comment 9•6 years ago
|
||
Should you be checking attributes["data-l10n-args"] before accessing it here: https://hg.mozilla.org/integration/autoland/rev/28805dd1b97a#l3.809 + if (attributes["data-l10n-id"]) { + document.l10n.setAttributes(elt, + attributes["data-l10n-id"], + attributes["data-l10n-args"]); + delete attributes["data-l10n-id"]; + if (attributes["data-l10n-args"]) { + delete attributes["data-l10n-args"]; + } We're porting this to TB and getting: ReferenceError: reference to undefined property "data-l10n-args" That's with dom.report_all_js_exceptions and/or javascript.options.strict set. Oops, can't set NI since it's already set.
Assignee | ||
Comment 10•6 years ago
|
||
(In reply to Jorg K (GMT+1) (urgent reviews and bustage fix only, Dec 22nd to Jan 1st) from comment #9) > Should you be checking attributes["data-l10n-args"] before accessing it here: > https://hg.mozilla.org/integration/autoland/rev/28805dd1b97a#l3.809 > > + if (attributes["data-l10n-id"]) { > + document.l10n.setAttributes(elt, > + attributes["data-l10n-id"], > + attributes["data-l10n-args"]); > + delete attributes["data-l10n-id"]; > + if (attributes["data-l10n-args"]) { > + delete attributes["data-l10n-args"]; > + } > > We're porting this to TB and getting: > ReferenceError: reference to undefined property "data-l10n-args" > That's with dom.report_all_js_exceptions and/or javascript.options.strict > set. > > Oops, can't set NI since it's already set. I wasn't testing with javascript.options.strict, and this code works fine with undefined being returned from the getter. I'll make the change to prevent the ReferenceError.
Flags: needinfo?(jaws)
Comment 11•6 years ago
|
||
Pushed by jwein@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/3f16eb2a8603 Convert about:support to Fluent. r=Gijs,flod https://hg.mozilla.org/integration/autoland/rev/32fe57d45d1b Use data-l10n-id instead of formatValue where the change is relatively straightforward. r=Gijs https://hg.mozilla.org/integration/autoland/rev/c68ba2c62949 Test fixes. r=flod,Gijs
Comment 12•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/3f16eb2a8603 https://hg.mozilla.org/mozilla-central/rev/32fe57d45d1b https://hg.mozilla.org/mozilla-central/rev/c68ba2c62949
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
status-firefox66:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla66
You need to log in
before you can comment on or make changes to this bug.
Description
•