The default bug view has changed. See this FAQ.

GCLI needs a addon, resize, restart, cookie and pagemanip commands

RESOLVED FIXED in Firefox 16

Status

()

Firefox
Developer Tools
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: miker, Unassigned)

Tracking

unspecified
Firefox 16
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [gclicommands])

Attachments

(1 attachment, 2 obsolete attachments)

Created attachment 639699 [details] [diff] [review]
Added addon, responsivemode & resizepage
Created attachment 640560 [details] [diff] [review]
More commands added

Current contents:
- addon
- resize
- restart
- pagemanip
- cookie
Attachment #639699 - Attachment is obsolete: true
Comment on attachment 640560 [details] [diff] [review]
More commands added

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

Also, we'll need to alter the patch message to reflect the title of the bug.

::: browser/devtools/commandline/GcliCommands.jsm
@@ +307,5 @@
>      threadManager.mainThread.dispatch({
>        run: function() {
>          hud.gcliterm.clearOutput();
>        }
> +    }, Components.interfaces.nsIThread.DISPATCH_NORMAL);

I'm not sure where the Ci->Components.interfaces came from, but I don't think we should be doing that. Happens in several places in this file.

@@ -307,4 @@
>    }
>  });
>  
> -

There's quite a bit of adding and removing blank lines. It's a bit of a nit, but unless there is a reason for it, we should avoid it, and since we're in this patch fixing the Ci/etc and the '.', we should do less LF churn IMHO.

::: browser/locales/en-US/chrome/browser/devtools/gclicommands.properties
@@ +398,4 @@
>  # LOCALIZATION NOTE (cmdDesc) A very short description of the 'cmd'
>  # command. This string is designed to be shown in a menu alongside the command
>  # name, which is why it should be as short as possible.
> +cmdDesc=Manipulate the commands.

Some of the descriptions have periods/fullstops, and some don't. So we need a standard. I vote no periods/fullstops because that's normal in menus.

Makes sense?
Attachment #640560 - Flags: review+
Summary: GCLI needs a master patch with all commands in one → GCLI needs a addon, resize, restart, cookie and pagemanip commands
Created attachment 640648 [details] [diff] [review]
Patch

(In reply to Joe Walker from comment #2)
> Comment on attachment 640560 [details] [diff] [review]
> More commands added
> 
> Review of attachment 640560 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> Also, we'll need to alter the patch message to reflect the title of the bug.
> 

Done

> ::: browser/devtools/commandline/GcliCommands.jsm
> @@ +307,5 @@
> >      threadManager.mainThread.dispatch({
> >        run: function() {
> >          hud.gcliterm.clearOutput();
> >        }
> > +    }, Components.interfaces.nsIThread.DISPATCH_NORMAL);
> 
> I'm not sure where the Ci->Components.interfaces came from, but I don't
> think we should be doing that. Happens in several places in this file.
> 

Changed

> @@ -307,4 @@
> >    }
> >  });
> >  
> > -
> 
> There's quite a bit of adding and removing blank lines. It's a bit of a nit,
> but unless there is a reason for it, we should avoid it, and since we're in
> this patch fixing the Ci/etc and the '.', we should do less LF churn IMHO.
> 

Agreed, removed.

> ::: browser/locales/en-US/chrome/browser/devtools/gclicommands.properties
> @@ +398,4 @@
> >  # LOCALIZATION NOTE (cmdDesc) A very short description of the 'cmd'
> >  # command. This string is designed to be shown in a menu alongside the command
> >  # name, which is why it should be as short as possible.
> > +cmdDesc=Manipulate the commands.
> 
> Some of the descriptions have periods/fullstops, and some don't. So we need
> a standard. I vote no periods/fullstops because that's normal in menus.
> 

Done

> Makes sense?

Ja, alles klar
Attachment #640560 - Attachment is obsolete: true
Whiteboard: [gclicommands] → [gclicommands][land-in-fx-team]
https://tbpl.mozilla.org/?tree=Fx-Team&rev=74218d787b06
Whiteboard: [gclicommands][land-in-fx-team] → [gclicommands][fixed-in-fx-team]
https://hg.mozilla.org/mozilla-central/rev/74218d787b06
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Whiteboard: [gclicommands][fixed-in-fx-team] → [gclicommands]
Target Milestone: --- → Firefox 16
https://hg.mozilla.org/mozilla-central/diff/74218d787b06/dom/base/nsFocusManager.cpp

:( Can you please revert that change? My non-debug build console is getting spammed.
Nevermind, I just reverted it:

https://hg.mozilla.org/mozilla-central/rev/d070bd19c526
(In reply to Tim Taubert [:ttaubert] from comment #7)
> Nevermind, I just reverted it:
> 
> https://hg.mozilla.org/mozilla-central/rev/d070bd19c526

Thanks, I was hunting this down :)
addonListUnknownHeading=The following addons of the selected type are currently installed:

Not sure about this string: "selected type" and "unknownheading"? When is this label used?
pagemodRemoveAttributeRootDesc=CSS selector of root of search
pagemodRemoveElementRootDesc=CSS selector specifying root of search
pagemodReplaceRootDesc=CSS selector to root of search

pagemodRemoveElementSearchDesc=CSS selector specifying elements to remove
pagemodRemoveAttributeSearchElementsDesc=CSS selector of elements to include

Shouldn't these strings use the same structure? If the answer is yes there are probably other strings with the same problem.


Elements matched by selector: %1$S. Replaces in text nodes: %2$S. Replaces in attributes: %3$S.

Replaces->Replacements?



Restarting Firefox...
Should use single unicode character instead of "..."
You need to log in before you can comment on or make changes to this bug.