ID for change.synckey.acceptButton has not been changed

VERIFIED FIXED in mozilla9

Status

Cloud Services
Firefox Sync: UI
VERIFIED FIXED
7 years ago
7 years ago

People

(Reporter: Alexander L. Slovesnik, Assigned: philikon)

Tracking

unspecified
mozilla9
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

7 years ago
From Bug 656492:

> -change.synckey.acceptButton = Change Sync Key
> +change.synckey.acceptButton = Change Recovery Key

ID for change.synckey.acceptButton in syncGenericChange.properties has not been changed. It should be changed for l10n purposes.
It's changed in change.recovery.acceptButton in syncGenericChange.js so I suspect that now UI is broken.
Even worse, the code that uses the string was updated, but to a nonsensical string ID. Should've spotted this in review, my bad.
Assignee: nobody → philipp
Created attachment 555269 [details] [diff] [review]
v1

There was one more string ID that wasn't correct. Also removed an unused string.
Attachment #555269 - Flags: review?(rnewman)
Comment on attachment 555269 [details] [diff] [review]
v1

Review of attachment 555269 [details] [diff] [review]:
-----------------------------------------------------------------

m-i! m-i!
Attachment #555269 - Flags: review?(rnewman) → review+
http://hg.mozilla.org/mozilla-central/rev/198c7de0699d
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla9
STRs for QA:

Apart from the fact that the string ID wasn't renamed, there were also two bugs concerning the the button and the explanatory text in the "My Sync Key" dialog. Those labels weren't updated and an exception occurred in the Error Console. These were fixed.
verified on m-c
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.