Closed
Bug 1478819
Opened 6 years ago
Closed 6 years ago
Doorhanger menu should have title case capitalization
Categories
(DevTools :: General, defect, P3)
Tracking
(firefox63 verified)
VERIFIED
FIXED
Firefox 63
Tracking | Status | |
---|---|---|
firefox63 | --- | verified |
People
(Reporter: victoria, Assigned: birtles)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
Assignee | ||
Comment 1•6 years ago
|
||
For my own reference, Localization content best practices[1] says: "If your changes are relevant only for English — for example, to correct a typographical error or to make letter case consistent — then there is generally no need to update the entity name." So it seems like we don't need to rev the string IDs for this. (Although it might be nice to invalidate, say, German, but not Japanese.) [1] https://developer.mozilla.org/en-US/docs/Mozilla/Localization/Localization_content_best_practices#Changing_existing_strings
Assignee | ||
Comment 2•6 years ago
|
||
(I should probably fix bug 1473211 first, however, and I'm not sure how well glob's review submission tool works for interdependent patches so I might have to land that first.)
Assignee | ||
Comment 3•6 years ago
|
||
As per: https://design.firefox.com/photon/copy/capitalization.html#platform-styles
Assignee | ||
Updated•6 years ago
|
Assignee: nobody → bbirtles
Status: NEW → ASSIGNED
Comment 4•6 years ago
|
||
Comment on attachment 8996600 [details] Bug 1478819 - Make Meatball Menu items use title case; r?jdescottes Francesco Lodolo [:flod] has approved the revision. https://phabricator.services.mozilla.com/D2591
Attachment #8996600 -
Flags: review+
Assignee | ||
Comment 5•6 years ago
|
||
Thanks Francesco!
Pushed by bbirtles@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/60b1a28da655 Make Meatball Menu items use title case; r=flod
Comment 7•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/60b1a28da655
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
status-firefox63:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 63
Updated•6 years ago
|
Flags: qe-verify+
Comment 8•6 years ago
|
||
This is verified fixed using Firefox 63.0b7 (BuildId:20180917143811) on Windows 10 64bit, macOS 10.13.6 and Ubuntu 16.04 64bit.
You need to log in
before you can comment on or make changes to this bug.
Description
•