Closed
Bug 1377847
Opened 7 years ago
Closed 7 years ago
Add localization comments to UI Density strings
Categories
(Firefox :: Toolbars and Customization, enhancement)
Tracking
()
RESOLVED
FIXED
Firefox 56
Tracking | Status | |
---|---|---|
firefox56 | --- | fixed |
People
(Reporter: flod, Assigned: flod)
Details
Attachments
(1 file)
Compact and Touch without context are confusing, since they can be either a verb or an adjective
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment 3•7 years ago
|
||
mozreview-review |
Comment on attachment 8882992 [details] Bug 1377847 - Add localization comments to UI Density strings https://reviewboard.mozilla.org/r/153956/#review159372 Thanks, that seems useful. Do you consider "Normal" easy enough to translate without context?
Attachment #8882992 -
Flags: review?(jhofmann) → review+
Assignee | ||
Comment 4•7 years ago
|
||
(In reply to Johann Hofmann [:johannh] from comment #3) > Thanks, that seems useful. Do you consider "Normal" easy enough to translate > without context? Good point. Unlike compact/touch there's no possible confusion on verb vs noun, but it won't hurt to add a comment to explain that it's part of the density menu.
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Pushed by francesco.lodolo@mozillaitalia.org: https://hg.mozilla.org/integration/autoland/rev/9d6d68c8f601 Add localization comments to UI Density strings r=johannh
Comment 8•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/9d6d68c8f601
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox56:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 56
You need to log in
before you can comment on or make changes to this bug.
Description
•