Closed
Bug 932822
Opened 11 years ago
Closed 5 years ago
L10N string "eventNodes" needs pluralization
Categories
(DevTools :: Debugger, defect, P3)
DevTools
Debugger
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: JasnaPaka, Unassigned)
References
(Blocks 1 open bug)
Details
In the file debugger.properties there is:
# LOCALIZATION NOTE (eventNodes): The text to display in the events tab when
# an event is listened on more than one target node.
eventNodes=%S nodes
I think we needs pluralization here.
Comment 1•11 years ago
|
||
(In reply to Pavel Cvrcek (Mozilla.cz) [:JasnaPaka] from comment #0)
>
> I think we needs pluralization here.
Good catch, but "nodes" will always be >= 2.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → INVALID
Reporter | ||
Comment 2•11 years ago
|
||
(In reply to Victor Porof [:vp] from comment #1)
> Good catch, but "nodes" will always be >= 2.
There are many plural forms around the world. See:
https://developer.mozilla.org/en/docs/Localization_and_Plurals
In this example we have "%S nodes" and for Czech translation it means:
2 - 4: %s uzly
5>=: %s uzlů
Status: RESOLVED → REOPENED
Resolution: INVALID → ---
Comment 3•11 years ago
|
||
Argh, I missed this one. String was introduced in bug 800857.
Victor: Pavel is right, we need a proper plural form in this case.
Comment 4•11 years ago
|
||
(In reply to Pavel Cvrcek (Mozilla.cz) [:JasnaPaka] from comment #2)
> (In reply to Victor Porof [:vp] from comment #1)
> > Good catch, but "nodes" will always be >= 2.
>
> There are many plural forms around the world. See:
> https://developer.mozilla.org/en/docs/Localization_and_Plurals
Oooh! Today I learned :) Sorry for closing as invalid.
Updated•11 years ago
|
Priority: -- → P3
Comment 5•10 years ago
|
||
(In reply to Pavel Cvrcek (Mozilla.cz) [:JasnaPaka] from comment #0)
> In the file debugger.properties there is:
>
> # LOCALIZATION NOTE (eventNodes): The text to display in the events tab when
> # an event is listened on more than one target node.
> eventNodes=%S nodes
>
> I think we needs pluralization here.
Pavel, would you be willing to fix this?
Updated•10 years ago
|
Blocks: dbg-frontend
Updated•6 years ago
|
Product: Firefox → DevTools
Comment 6•6 years ago
|
||
This string is no longer used but, as we'll be implementing event breakpoints in the debugger very soon, I'll leave and see if it still applies. Will evaluate after when the event listener feature is complete.
Updated•6 years ago
|
Status: REOPENED → NEW
Comment 7•5 years ago
|
||
This string was removed so I'm closing this.
Status: NEW → RESOLVED
Closed: 11 years ago → 5 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•