Closed
Bug 921115
Opened 11 years ago
Closed 11 years ago
[Clock] Lap string is not localized
Categories
(Firefox OS Graveyard :: Gaia::Clock, defect)
Tracking
(blocking-b2g:koi+, b2g-v1.2 fixed)
People
(Reporter: jugglinmike, Assigned: evhan55)
Details
(Keywords: late-l10n)
Attachments
(2 files)
Steps to reproduce: 1. Change the system language in the Settings application to a value other than "English" 2. Open the Clock application 3. Navigate to the "Stopwatch" panel 4. Start the stopwatch 5. Press the "Lap" button Expected result: A new entry is added to the list of laps describing the lap in the selected language. Actual result: A new entry is added to the list of laps describing the lap in English.
Reporter | ||
Comment 1•11 years ago
|
||
Requesting "koi+" because the application cannot be fully localized in its present state.
blocking-b2g: --- → koi?
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → evelyn
Assignee | ||
Comment 2•11 years ago
|
||
- stopwatch_panel.js - Use navigator.mozL10n.localize to localize the 'Lap [n]' label - Had to add an id to the element that is set via a template to be able to use it in the localize method - stopwatch_panel_test.js - I think a stopwatch_panel_test.js is already in another PR (https://github.com/mozilla-b2g/gaia/pull/12454), so this one is submitted here purely for review
Attachment #812633 -
Flags: review?(mike)
Assignee | ||
Comment 3•11 years ago
|
||
Comment on attachment 812633 [details] [review] https://github.com/mozilla-b2g/gaia/pull/12579 Removed the lap-number #id for a simpler approach
Assignee | ||
Comment 4•11 years ago
|
||
Comment on attachment 812633 [details] [review] https://github.com/mozilla-b2g/gaia/pull/12579 Cancelling until the newly rebased version of this patch shows up on GitHub
Attachment #812633 -
Flags: review?(mike)
Assignee | ||
Comment 5•11 years ago
|
||
Comment on attachment 812633 [details] [review] https://github.com/mozilla-b2g/gaia/pull/12579 Reopening for review after re-basing with master: - stopwatch_panel.js - Use navigator.mozL10n.localize to localize the 'Lap [n]' label - stopwatch_panel_test.js - Test localization in general with 'Lap localization' test - Change previous direct tests for 'Lap 1' string, etc., with tests against calls to localize instead
Attachment #812633 -
Flags: review?(mike)
Reporter | ||
Comment 6•11 years ago
|
||
Comment on attachment 812633 [details] [review] https://github.com/mozilla-b2g/gaia/pull/12579 I've requested two minor changes on the pull request on GitHub. Other than that, this looks good! I'm happy with how precise the changes are, and I appreciate you being so proactive with rebasing over Yannick's recent commit :)
Attachment #812633 -
Flags: review?(mike)
Assignee | ||
Comment 7•11 years ago
|
||
Comment on attachment 812633 [details] [review] https://github.com/mozilla-b2g/gaia/pull/12579 Resubmitting for review!
Attachment #812633 -
Flags: review?(mike)
Reporter | ||
Comment 8•11 years ago
|
||
Comment on attachment 812633 [details] [review] https://github.com/mozilla-b2g/gaia/pull/12579 Looks good! I'll merge this in just a moment
Attachment #812633 -
Flags: review?(mike) → review+
Reporter | ||
Comment 9•11 years ago
|
||
master: https://github.com/mozilla-b2g/gaia/commit/8c1a5f2205445bcb51bcc8406b38bf7b6b0cee43 Thanks, Evelyn
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Comment 10•11 years ago
|
||
triage: fix to enable localization of a prominent new feature for Clock in 1.2, feels like we need to block on this one.
blocking-b2g: koi? → koi+
Updated•11 years ago
|
Attachment mime type: text/plain → text/x-github-pull-request
Comment 11•11 years ago
|
||
Uplifted 8c1a5f2205445bcb51bcc8406b38bf7b6b0cee43 to: v1.2: 3de512631816c14e88c9649ed35545fceac1fcab
status-b2g-v1.2:
--- → fixed
Comment 12•11 years ago
|
||
Thanks! Verified it. * Test Build: - Gaia: 2ef9bc3c7a6de228b63e6ba3613eb0c0dd639c59 - Gecko: http://hg.mozilla.org/releases/mozilla-aurora/rev/4a94d2ea9d37 - BuildID 20131028004002 - Version 26.0a2 Attaching the screenshot.
Status: RESOLVED → VERIFIED
Comment 13•11 years ago
|
||
You need to log in
before you can comment on or make changes to this bug.
Description
•