Closed Bug 990606 Opened 7 years ago Closed 7 years ago

[Camera][Madai][l10n] Improve l10n for Camera Settings

Categories

(Firefox OS Graveyard :: Gaia::Camera, defect)

defect
Not set
normal

Tracking

(blocking-b2g:1.4+, b2g-v1.4 fixed, b2g-v2.0 fixed)

RESOLVED FIXED
1.4 S5 (11apr)
blocking-b2g 1.4+
Tracking Status
b2g-v1.4 --- fixed
b2g-v2.0 --- fixed

People

(Reporter: tchevalier, Assigned: wilsonpage)

References

Details

(Keywords: l12y, late-l10n)

Attachments

(2 files)

Attached image 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

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?
blocking-b2g: 1.4? → 1.4+
Flags: needinfo?(wilsonpage)
Flags: needinfo?(dflanagan)
Keywords: late-l10n
Attached file pull-request (master)
Attachment #8401783 - Flags: review?(tchevalier)
Attachment #8401783 - Flags: review?(jdarcangelo)
Attachment #8401783 - Flags: review?(francesco.lodolo)
Flags: needinfo?(wilsonpage)
Assignee: nobody → wilsonpage
flod: Could you re-review?
Flags: needinfo?(francesco.lodolo)
Comment on attachment 8401783 [details]
pull-request (master)

Looks good with a couple of comments on Github.
Attachment #8401783 - Flags: review?(francesco.lodolo) → feedback+
Flags: needinfo?(francesco.lodolo)
Attachment #8401783 - Flags: review?(jdarcangelo) → review+
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Flags: needinfo?(dflanagan)
Attachment #8401783 - Flags: review?(tchevalier)
Not enough information with current STR to write test case to address bug.
QA Whiteboard: [QAnalyst-Triage?]
Flags: needinfo?(ktucker)
Flags: in-moztrap?(rmitchell)
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+]
Flags: needinfo?(ktucker)
Flags: in-moztrap?(rmitchell)
Flags: in-moztrap+
You need to log in before you can comment on or make changes to this bug.