Closed Bug 1233189 Opened 9 years ago Closed 9 years ago

Duplicated string first_time_experience_button_label in loop.properties

Categories

(Hello (Loop) :: Client, defect, P1)

defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: flod, Assigned: crafuse)

References

Details

Attachments

(1 file)

http://hg.mozilla.org/mozilla-central/diff/ec836d27b563/browser/locales/en-US/chrome/browser/loop/loop.properties There was already such string with a different value first_time_experience_button_label=Get Started Considering that, I expect English to not show the correct text either.
NI for visibility, even if we're at the very beginning of the cycle.
Flags: needinfo?(crafuse)
Dan, Does this need to happen in gecko-dev/Mozilla Central? REMOVE: first_time_experience_button_label=Get Started Increment and translate for: first_time_experience_button_label=See how it works TO: first_time_experience_button_label2=See how it works Change to: add-on/panels/js/panel.jsx 38: caption={mozL10n.get("first_time_experience_button_label2")}
Flags: needinfo?(crafuse) → needinfo?(dmose)
I have mixed feelings on this one. Deferring to Standard8 in case he has a strong opinion...
Flags: needinfo?(standard8)
As this is broken, lets get it fixed on m-c so that its better for localisers for now.
Flags: needinfo?(standard8)
Assignee: nobody → crafuse
Status: NEW → ASSIGNED
FWIW, this issue causes a warning to be spammed to the terminal, too (during startup on debug builds at least): WARNING: the property first_time_experience_button_label already exists : file ../../../mozilla/xpcom/ds/nsPersistentProperties.cpp, line 532
Comment on attachment 8699777 [details] [diff] [review] Duplicated string first_time_experience_button_label in loop.properties Removed string property and modified display location. Needs to eventually be synced with loop master.
Attachment #8699777 - Flags: review?(standard8)
Comment on attachment 8699777 [details] [diff] [review] Duplicated string first_time_experience_button_label in loop.properties Looks good. r=Standard8
Attachment #8699777 - Flags: review?(standard8) → review+
Flags: needinfo?(dmose)
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: