Move whyPaused.* strings to Fluent
Categories
(DevTools :: Debugger, task)
Tracking
(firefox96 fixed)
Tracking | Status | |
---|---|---|
firefox96 | --- | fixed |
People
(Reporter: nchevobbe, Assigned: nchevobbe)
References
(Blocks 1 open bug)
Details
Attachments
(2 files)
Assignee | ||
Comment 1•1 year ago
|
||
Since those strings can be consumed by the PausedDebuggerOverlay, on the server,
we need to put them in devtools/shared.
As this will create some work for the l10n team, we migrate them to Fluent so we're
feeding 2 birds with 1 scone.
The <LocalizationProvider>
is added directly in <WhyPaused>
(and not in <App>
,
as it's messing up with the React context (See Bug 1743155).
Some snapshots are updated. It's unfortunate that we don't see the translated
strings there anymore, but we can't have nice things in Jest.
Assignee | ||
Comment 2•1 year ago
|
||
While working around those strings, I realized we didn't
check the text of the component in the different situations
it handles, so I added a few assertions.
Depends on D132260
Pushed by nchevobbe@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/fd097d484cfb [devtools] Move whyPaused.* string to a ftl file in devtools/shared. r=bomsy,jdescottes,fluent-reviewers,flod https://hg.mozilla.org/integration/autoland/rev/cf7717dfbd7c [devtools] Check whyPaused text in mochitests. r=bomsy.
Comment 4•1 year ago
|
||
Backed out for causing mochitest failures on browser_metrics_debugger.js.
Backout link: https://hg.mozilla.org/integration/autoland/rev/c6b7e343048bd01cfd8f5df563795debcd558fdf
Link to failure log : https://treeherder.mozilla.org/logviewer?job_id=359928129&repo=autoland&lineNumber=2865
Failure message: TEST-UNEXPECTED-FAIL | devtools/client/framework/test/metrics/browser_metrics_debugger.js | Duplicated module load detected. List of duplicated modules
Pushed by nchevobbe@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/6eaac86a4922 [devtools] Move whyPaused.* string to a ftl file in devtools/shared. r=bomsy,jdescottes,fluent-reviewers,flod https://hg.mozilla.org/integration/autoland/rev/41993db6c7a1 [devtools] Check whyPaused text in mochitests. r=bomsy.
Comment 6•1 year ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/6eaac86a4922
https://hg.mozilla.org/mozilla-central/rev/41993db6c7a1
Description
•