|Submitter||Diff||Changes||Open Issues||Last Updated|
|Error loading review requests:|
59 bytes, text/x-review-board-request
|Details | Review|
We plan to run netmonitor on both toolbar and browser tab. * Write a l10n properties loader * Support L10N.numberWithDecimals * Avoid plural-form.js, intl.properties, l10n.js duplication
Assignee: nobody → rchien
Status: NEW → ASSIGNED
Support L10N.numberWithDecimals in devtools-core has done in https://github.com/devtools-html/devtools-core/pull/275.
After having a discussion with Jason, we agreed that move http://searchfox.org/mozilla-central/source/devtools/shared/l10n.js to devtools-core/packages/devtools-module. PR is coming soon.
PR in devtools-core https://github.com/devtools-html/devtools-core/pull/298
L10N done! After fixing L10N issues, launchpad can be bootstrapped properly and see the landing page. Next step will fix connection timeout.
Comment on attachment 8852789 [details] Bug 1350218 - Support L10N properties loader in Launchpad https://reviewboard.mozilla.org/r/124962/#review127700 I have nothing to add, looks good to me. R+ Thanks Ricky! Honza
Attachment #8852789 - Flags: review?(odvarko) → review+
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/0896cb720b91 Support L10N properties loader in Launchpad r=Honza
Status: ASSIGNED → RESOLVED
Last Resolved: a year ago
status-firefox55: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 55
You need to log in before you can comment on or make changes to this bug.