Closed Bug 785714 Opened 12 years ago Closed 12 years ago

gclicommand |help jsb| contains hardcoded string for url

Categories

(DevTools :: Console, defect)

16 Branch
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 18

People

(Reporter: aryx, Assigned: ananuti)

Details

(Whiteboard: [gclicommands][fixed-in-fx-team])

Attachments

(1 file, 1 obsolete file)

The following string in GcliCommans.jsm should be localized:

{
  name: 'url',
  type: 'string',
  description: gcli.lookup('jsbUrlDesc'),
  manual: 'The URL of the JS to prettify'
},
Whiteboard: [gclicommands]
Attached patch patch (obsolete) — Splinter Review
Assignee: nobody → ananuti
Status: NEW → ASSIGNED
Attachment #655543 - Flags: review?(dcamp)
Comment on attachment 655543 [details] [diff] [review]
patch

Review of attachment 655543 [details] [diff] [review]:
-----------------------------------------------------------------

GCLI automatically uses the 'description' field when there isn't a 'manual' entry, so supplying both with the same string doesn't help. I suggest in this case that we just remove the entry for the manual field.
Thanks.
Comment on attachment 655543 [details] [diff] [review]
patch

Review of attachment 655543 [details] [diff] [review]:
-----------------------------------------------------------------

::: browser/locales/en-US/chrome/browser/devtools/gclicommands.properties
@@ +740,5 @@
>  
> +# LOCALIZATION NOTE (jsbUrlManual) A fuller description of the
> +# 'jsb <url>' parameter, displayed when the user asks for help on what it
> +# does.
> +jsbUrlManual=The URL of the JS to prettify

If we're not removing that key, this should be consistently 'beautify'.
(In reply to Axel Hecht [:Pike] from comment #3)
> If we're not removing that key, this should be consistently 'beautify'.

This will be fixed by bug 782820.
Attachment #655543 - Attachment is obsolete: true
Attachment #655543 - Flags: review?(dcamp)
Attached patch patchSplinter Review
rm manual string for jsb url.
Attachment #655829 - Flags: review?(jwalker)
Comment on attachment 655829 [details] [diff] [review]
patch

Review of attachment 655829 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks.
Will add this to my landing queue
Attachment #655829 - Flags: review?(jwalker) → review+
https://tbpl.mozilla.org/?tree=Fx-Team&rev=3636c402ea3d
Whiteboard: [gclicommands] → [gclicommands][fixed-in-fx-team]
https://hg.mozilla.org/mozilla-central/rev/3ac34e8c7c9b
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 18
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: