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)
Hello (Loop)
Client
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: flod, Assigned: crafuse)
References
Details
Attachments
(1 file)
(deleted),
patch
|
standard8
:
review+
|
Details | Diff | Splinter Review |
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.
Reporter | ||
Comment 1•9 years ago
|
||
NI for visibility, even if we're at the very beginning of the cycle.
Flags: needinfo?(crafuse)
Assignee | ||
Comment 2•9 years ago
|
||
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)
Comment 3•9 years ago
|
||
I have mixed feelings on this one. Deferring to Standard8 in case he has a strong opinion...
Flags: needinfo?(standard8)
Comment 4•9 years ago
|
||
As this is broken, lets get it fixed on m-c so that its better for localisers for now.
Flags: needinfo?(standard8)
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → crafuse
Assignee | ||
Updated•9 years ago
|
Status: NEW → ASSIGNED
Comment 5•9 years ago
|
||
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
Assignee | ||
Comment 6•9 years ago
|
||
Assignee | ||
Comment 7•9 years ago
|
||
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 8•9 years ago
|
||
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+
Updated•9 years ago
|
Flags: needinfo?(dmose)
Assignee | ||
Updated•9 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Comment 10•9 years ago
|
||
bugherder |
You need to log in
before you can comment on or make changes to this bug.
Description
•