892268 - Ctrl/Cmd+0/+/- should change the text size in all the devtools and the panel

Status: RESOLVED FIXED

(DevTools :: General, defect)

Description

[Nick Fitzgerald [:fitzgen] [⏰PST; UTC-8]]

A few of the tools support this now in certain panes, but we should have this baked in for everything and also change the size of the toolbar text too.

Comment 1

[Heather Arthur [:harth]]

Attachment: zoom.patch

This patch patch add Ctrl+/-/0 to the toolbox, and removes those bindings from the web console.

It also focuses a devtools tab when it's selected. Before you'd have to click on a text area to focus the toolbox (and thus couldn't use our tab switching or zooming shortcuts until then).

Try: https://tbpl.mozilla.org/?tree=Try&rev=f121bd778f93

Comment 2

[Mihai Sucan [:msucan]]

Comment on attachment 804083 [details] [diff] [review]
zoom.patch

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

Whole-hearted r+! Just tested and it makes my day. *Very* glad to see this full zoom for the toolbox. Now everything is much more readable for me. Thank you Heather!

One concern: the removal of zoom support from the Web Console makes sense, but the Browser Console is not part of the toolbox yet. Can you please keep zoom in the Browser Console only? Until we have the Browser toolbox.

Comment 3

[Heather Arthur [:harth]]

Mihai, can you give(In reply to Mihai Sucan [:msucan] from comment #2)
> One concern: the removal of zoom support from the Web Console makes sense,
> but the Browser Console is not part of the toolbox yet. Can you please keep
> zoom in the Browser Console only? Until we have the Browser toolbox.

Any quick code pointers on how to do that?

Comment 4

[Mihai Sucan [:msucan]]

(In reply to Heather Arthur [:harth] from comment #3)
> Mihai, can you give(In reply to Mihai Sucan [:msucan] from comment #2)
> > One concern: the removal of zoom support from the Web Console makes sense,
> > but the Browser Console is not part of the toolbox yet. Can you please keep
> > zoom in the Browser Console only? Until we have the Browser toolbox.
> 
> Any quick code pointers on how to do that?

Browser Console instances have a _browserConsole boolean flag set to true. In the command controller I suggest you return false for isCommandEnabled() and for supportsCommand(), when the flag is false. See how cmd_close works.

Comment 5

[Heather Arthur [:harth]]

(In reply to Mihai Sucan [:msucan] from comment #4)
> Browser Console instances have a _browserConsole boolean flag set to true.
> In the command controller I suggest you return false for isCommandEnabled()
> and for supportsCommand(), when the flag is false. See how cmd_close works.

Thanks for the tip. Unfortunately that makes it so the web console still swallows those keys but does nothing. Barring any breakthroughs soon I'll just add back the web console zoom and have that one be weird.

Comment 6

[Heather Arthur [:harth]]

https://hg.mozilla.org/integration/fx-team/rev/e50ef9cabb1f

Comment 7

[Mihai Sucan [:msucan]]

Attachment: keep-browser-console-zoom

Heather: this is what I had in mind for the webconsole.js/xul changes. Like in the case of cmd_close: supportsCommand returns false, cmd_fullZoom commands are disabled by default in the xul file, and during init, if it is a browser console, we remove the 'disabled' attribute from the zoom commands.

(I started to make this patch by first reverting the changes to webconsole.jss/xul/dtd from your patch.)

Since your patch already landed, I am not sure how we should proceed. Should we make a patch that applies for fx-team or should we open a new bug?

Comment 8

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/e50ef9cabb1f

Comment 9

[Robert Nyman]

Cmd and + and Cmd and - works for me in Mac OS X, but Cmd and 0 does nothing (i.e. doesn't restore it to the original size).

Comment 10

[Heather Arthur [:harth]]

(In reply to Robert Nyman from comment #9)
> Cmd and + and Cmd and - works for me in Mac OS X, but Cmd and 0 does nothing
> (i.e. doesn't restore it to the original size).

Works for me. Make sure the devtools has focus by clicking on a tool tab. File a new bug about it.

Comment 11

[Robert Nyman]

Apparently it worked after a restart of Aurora. Thanks.