Closed Bug 990606 Opened 7 years ago Closed 7 years ago
[Camera][Madai][l10n] Improve l10n for Camera Settings
A couple of issues for l10n which needs to be fixed on 1.4 if Madai lands there too. 1/ The following strings are used as label and title, we should use different strings (see bug 950463): hdr = HDR grid = Grid flash = Flash self-timer = Self-timer Please add new strings for title like, hdr-header = HDR, etc. 2/ As you can see on the attached screenshot, for labels on the left screen (the same as in the first issue), the first letter of each word is automatically uppercased ("du capteur photo" vs. "Du Capteur Photo"). It shouldn't. Note, I used a fake translation to test things out.
(In reply to Théo Chevalier [:tchevalier] from comment #0) > Created attachment 8400027 [details] > Screenshot > > A couple of issues for l10n which needs to be fixed on 1.4 if Madai lands > there too. > > 1/ > The following strings are used as label and title, we should use different > strings (see bug 950463): > hdr = HDR > grid = Grid > flash = Flash > self-timer = Self-timer Can't test, but of course we will need new strings for camera-resolution video-resolution scene-mode as well if they are used on both settings panel and header. However I think it is safe to reuse the label strings in the notification displayed when changing a setting.
Unfortunately I can't test it either at the moment, and I missed the string reuse from the screenshot in bug 988519 :-\ Looking at your screenshot, I think the header is broken too (it should be truncated before reaching the button, not overlapping).
Camera has been uplifted to 1.4 in https://github.com/mozilla-b2g/gaia/pull/17935, nominating this bug
blocking-b2g: --- → 1.4?
flod: Could you re-review?
Comment on attachment 8401783 [details] pull-request (master) Looks good with a couple of comments on Github.
Attachment #8401783 - Flags: review?(francesco.lodolo) → feedback+
Attachment #8401783 - Flags: review?(jdarcangelo) → review+
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Not enough information with current STR to write test case to address bug.
QA Whiteboard: [QAnalyst-Triage?]
Test cases are in moztrap: https://moztrap.mozilla.org/manage/case/12450/ https://moztrap.mozilla.org/manage/case/12454/ https://moztrap.mozilla.org/manage/case/12417/ https://moztrap.mozilla.org/manage/case/14346/
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
You need to log in before you can comment on or make changes to this bug.