Closed Bug 1377847 Opened 2 years ago Closed 2 years ago

Add localization comments to UI Density strings

Categories

(Firefox :: Toolbars and Customization, enhancement)

56 Branch
enhancement
Not set

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 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+
(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.
Pushed by francesco.lodolo@mozillaitalia.org:
https://hg.mozilla.org/integration/autoland/rev/9d6d68c8f601
Add localization comments to UI Density strings r=johannh
https://hg.mozilla.org/mozilla-central/rev/9d6d68c8f601
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 56
You need to log in before you can comment on or make changes to this bug.