Closed
Bug 1050085
Opened 10 years ago
Closed 9 years ago
Improve l10n comments
Categories
(DevTools :: Performance Tools (Profiler/Timeline), defect, P3)
DevTools
Performance Tools (Profiler/Timeline)
Tracking
(firefox42 fixed)
RESOLVED
FIXED
Firefox 42
Tracking | Status | |
---|---|---|
firefox42 | --- | fixed |
People
(Reporter: flod, Assigned: jsantell)
References
Details
Attachments
(1 file)
(deleted),
patch
|
flod
:
review+
|
Details | Diff | Splinter Review |
profiler.dtd
Given the structure profilerUI.emptyNotice1+2 and profilerUI.stopNotice1+2, I'd assume these strings are concatenated with a button in the middle, like it happens on the Network tab. The localization comment should clearly explain that.
Typo in comment for profilerUI.clearButton: "remvoes all the recordings"
profiler.properties
recordingsList.itemLabel=Recording #%S
What's %S? Given the #, I'd say a number, but I shouldn't be guessing it. There should be an explanation of what %S is in the comment, and an example.
Updated•10 years ago
|
Priority: -- → P3
Updated•10 years ago
|
Summary: [Profiler] Improve l10n comments → Improve l10n comments
Assignee | ||
Comment 1•10 years ago
|
||
Address comment issue in:
https://bugzilla.mozilla.org/show_bug.cgi?id=1160900#c11
Assignee | ||
Comment 2•10 years ago
|
||
Address comment issue in:
https://bugzilla.mozilla.org/show_bug.cgi?id=1082695#c24
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → jsantell
Assignee | ||
Updated•9 years ago
|
Assignee: jsantell → nobody
Assignee | ||
Updated•9 years ago
|
Blocks: perf-tools-fx42
Assignee | ||
Comment 3•9 years ago
|
||
Seems like most of these were taken care of already, but here are the remaining ones
Assignee: nobody → jsantell
Status: NEW → ASSIGNED
Attachment #8643413 -
Flags: review?(francesco.lodolo)
Reporter | ||
Comment 4•9 years ago
|
||
Comment on attachment 8643413 [details] [diff] [review]
1050085-l10n-cleanup.patch
Review of attachment 8643413 [details] [diff] [review]:
-----------------------------------------------------------------
Looks good, thanks.
Attachment #8643413 -
Flags: review?(francesco.lodolo) → review+
Comment 6•9 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
status-firefox42:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 42
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•