Migrate safeMode.dtd to Fluent

RESOLVED FIXED in Firefox 66

Status

()

P3
normal
RESOLVED FIXED
2 months ago
26 days ago

People

(Reporter: jaws, Assigned: nickcowles9575, Mentored)

Tracking

(Blocks: 1 bug)

Trunk
Firefox 66
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox66 fixed)

Details

Attachments

(1 attachment, 1 obsolete attachment)

safeMode.dtd[1] should be migrated to Fluent. The new Fluent file should be created at browser\locales\en-US\browser\safeMode.ftl

A migration file should also be created at python\l10n\fluent_migrations

The pageInfo.dtd includes a maxWidth that define the width of the subsequent dialog. This sets the maxwidth attribute on the dialog. Due to the asynchronous nature of Fluent, this value will need to be set on the style attribute instead. Please migrate the maxWidth value to a style attribute value as was done in [2]. [3] is an example migration script for converting from a width attribute to a style attribute.

[1] https://searchfox.org/mozilla-central/source/browser/locales/en-US/chrome/browser/safeMode.dtd
[2] https://searchfox.org/mozilla-central/rev/0ee0b63732d35d16ba22d5a1120622e2e8d58c29/browser/locales/en-US/browser/preferences/applicationManager.ftl#7
[3] https://hg.mozilla.org/integration/autoland/diff/ab889ec68ba2/python/l10n/fluent_migrations/bug_1451992_preferences_colors.py#l1.32
Priority: -- → P3

In the second paragraph of the above comment, I meant to refer to safeMode.dtd, not pageInfo.dtd.

Assignee: nobody → nickcowles9575
Status: NEW → ASSIGNED
(Assignee)

Comment 2

a month ago

Created attachment 9037585 [details]
Bug 1517529 - Migrate safeMode dialog to Fluent for localization, r?Gijs,jaws

(Assignee)

Comment 3

a month ago

Created attachment 9037595 [details]
Bug 1517529 - Migrate safeMode dialog to Fluent for localization, r?Gijs,jaws

Attachment #9037595 - Attachment is obsolete: true

Comment 4

a month ago
Pushed by gijskruitbosch@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/585816362855
Migrate safeMode dialog to Fluent for localization, r=Gijs,flod

Comment 5

a month ago
bugherder
Status: ASSIGNED → RESOLVED
Last Resolved: a month ago
status-firefox66: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 66
You need to log in before you can comment on or make changes to this bug.