Closed
Bug 799463
Opened 12 years ago
Closed 12 years ago
GCLI jsb command should have nice html-free updates to jsbBraceStyleDesc and jsbBraceStyleManual strings
Categories
(DevTools Graveyard :: Graphic Commandline and Toolbar, defect)
DevTools Graveyard
Graphic Commandline and Toolbar
Tracking
(Not tracked)
RESOLVED
FIXED
Firefox 22
People
(Reporter: jwalker, Assigned: miker)
References
Details
Attachments
(1 file, 2 obsolete files)
(deleted),
patch
|
Details | Diff | Splinter Review |
We'll need to change the string names, obviously.
Assignee | ||
Updated•12 years ago
|
Assignee: nobody → mratcliffe
Assignee | ||
Comment 1•12 years ago
|
||
Attachment #669579 -
Flags: review?(jwalker)
Comment 2•12 years ago
|
||
Comment on attachment 669579 [details] [diff] [review] Updated strings Review of attachment 669579 [details] [diff] [review]: ----------------------------------------------------------------- ::: browser/locales/en-US/chrome/browser/devtools/gclicommands.properties @@ +853,4 @@ > # 'jsb <braceStyle>' parameter. This string is designed to be shown > # in a menu alongside the command name, which is why it should be as short as > # possible. > +jsbBraceStyleDesc2=Select the coding style of braces There's probably a more descriptive change to the key, I suggest to think about them as variable names. How about jsbSelectBraceStyleDesc? @@ +859,3 @@ > # 'jsb <braceStyle>' parameter, displayed when the user asks for help > # on what it does. > +jsbBraceStyleManual2=Select the coding style of braces: collapse - put braces on the same line as control statements, expand - put braces on own line (Allman / ANSI style), end-expand - put end braces on own line or expand-strict - put brace on own line even if it will break your code. I'd recommend to get a copy writer for this like Matej. My non-native speaker suggestion would be to use ';' to separate the cases consistently (no or for the last one).
Reporter | ||
Comment 3•12 years ago
|
||
(In reply to Axel Hecht [:Pike] from comment #2) > Comment on attachment 669579 [details] [diff] [review] > Updated strings > > Review of attachment 669579 [details] [diff] [review]: > ----------------------------------------------------------------- > > ::: browser/locales/en-US/chrome/browser/devtools/gclicommands.properties > @@ +853,4 @@ > > # 'jsb <braceStyle>' parameter. This string is designed to be shown > > # in a menu alongside the command name, which is why it should be as short as > > # possible. > > +jsbBraceStyleDesc2=Select the coding style of braces > > There's probably a more descriptive change to the key, I suggest to think > about them as variable names. How about jsbSelectBraceStyleDesc? This idea here is that this is the 'desc'ription of the 'braceStyle' parameter to the 'jsb' command. All GCLI parameter strings follow this basic pattern (<command><parameter><use>), and we add numbers to the end to avoid breaking the pattern needlessly.
Assignee | ||
Comment 4•12 years ago
|
||
(In reply to Axel Hecht [:Pike] from comment #2) > Comment on attachment 669579 [details] [diff] [review] > Updated strings > > Review of attachment 669579 [details] [diff] [review]: > ----------------------------------------------------------------- > > ::: browser/locales/en-US/chrome/browser/devtools/gclicommands.properties > @@ +853,4 @@ > > # 'jsb <braceStyle>' parameter. This string is designed to be shown > > # in a menu alongside the command name, which is why it should be as short as > > # possible. > > +jsbBraceStyleDesc2=Select the coding style of braces > > There's probably a more descriptive change to the key, I suggest to think > about them as variable names. How about jsbSelectBraceStyleDesc? > This idea here is that this is the 'desc'ription of the 'braceStyle' parameter to the 'jsb' command. All GCLI parameter strings follow this basic pattern (<command><parameter><use>), and we add numbers to the end to avoid breaking the pattern needlessly. > @@ +859,3 @@ > > # 'jsb <braceStyle>' parameter, displayed when the user asks for help > > # on what it does. > > +jsbBraceStyleManual2=Select the coding style of braces: collapse - put braces on the same line as control statements, expand - put braces on own line (Allman / ANSI style), end-expand - put end braces on own line or expand-strict - put brace on own line even if it will break your code. > > I'd recommend to get a copy writer for this like Matej. > > My non-native speaker suggestion would be to use ';' to separate the cases > consistently (no or for the last one). Done
Attachment #669579 -
Attachment is obsolete: true
Attachment #669579 -
Flags: review?(jwalker)
Attachment #669902 -
Flags: review?(jwalker)
Reporter | ||
Comment 5•12 years ago
|
||
Comment on attachment 669902 [details] [diff] [review] Updated strings v2 Review of attachment 669902 [details] [diff] [review]: ----------------------------------------------------------------- I think we need to get bug 799462 landed, so we can get it in aurora fairly soon, so we don't confuse any more translators. This means that this bug will be bitrotted. r+ nevertheless.
Attachment #669902 -
Flags: review?(jwalker) → review+
Reporter | ||
Comment 6•12 years ago
|
||
New component triage. Filter on "Lobster Thermidor aux crevettes with a Mornay sauce"
Component: Developer Tools: Console → Developer Tools: Graphic Commandline and Toolbar
Assignee | ||
Updated•12 years ago
|
Blocks: DevToolsPaperCuts
Whiteboard: [land-in-fx-team]
Comment 8•12 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/b824eca042e8
Whiteboard: [land-in-fx-team] → [fixed-in-fx-team]
Comment 9•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/b824eca042e8
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 22
Comment 10•12 years ago
|
||
I think I need a little explanation here jsbBraceStyleManual2=Select the coding style of braces: collapse - put braces on the same line as control statements; expand - put braces on own line (Allman / ANSI style); end-expand - put end braces on own line; expand-strict - put braces on own line even if it will break your code. 1) My understanding is that collapse, end-expand, etc. should not be translated, since they're parameter values. Am I right? In this case a localization comment should be added to explain it. 2) "even if it will break your code". Breaking the code means that the code is no more functional or something else?
Assignee | ||
Comment 11•12 years ago
|
||
(In reply to Francesco Lodolo [:flod] from comment #10) > I think I need a little explanation here > > jsbBraceStyleManual2=Select the coding style of braces: collapse - put > braces on the same line as control statements; expand - put braces on own > line (Allman / ANSI style); end-expand - put end braces on own line; > expand-strict - put braces on own line even if it will break your code. > > 1) My understanding is that collapse, end-expand, etc. should not be > translated, since they're parameter values. Am I right? In this case a > localization comment should be added to explain it. > > 2) "even if it will break your code". Breaking the code means that the code > is no more functional or something else? This bug is already landed so I have logged bug 845750 to address your comments.
Reporter | ||
Updated•11 years ago
|
No longer blocks: DevToolsPaperCuts
Updated•6 years ago
|
Product: Firefox → DevTools
Updated•6 years ago
|
Product: DevTools → DevTools Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•