942686 - Pressing ESC in a docked scratchpad closes the sidebar but also opens the split console

Status: RESOLVED WONTFIX

(DevTools Graveyard :: Scratchpad, defect, P3)

Description

[Panos Astithas (he/him) [:past] (please ni?)]

I noticed this when playing with the App Manager, which meant I was using a remote scratchpad inside the toolbox. Typing window<Cmd-I> opens the variables view sidebar. Now hitting ESC removes the sidebar, but is also displaying the web console in split mode. We should prevent the event propagation in this case.

Comment 1

[Panos Astithas (he/him) [:past] (please ni?)]

Brian, is this an easy fix?

Comment 2

[Brian Grinstead [:bgrins]]

(In reply to Panos Astithas [:past] from comment #1)
> Brian, is this an easy fix?

Yes, I've had to do a few of these recently.  If the event is registered on keydown, then a stopPropagation should resolve it.  If it happening on keypress we may need to make the event use capturing instead of bubbling depending on which element the events are bound to.

When trying to test this, for some reason when running scratchpad in the toolbox for app manager, my keyboard shortcuts are not working at all.  Pressing Cmd-I opens up the 'page information' box, and escape is doing nothing.  Maybe this is just something weird in my environment - I take it the shortcuts working for you?

Comment 3

[Brian Grinstead [:bgrins]]

Panos,
I'm not able to reproduce this with docked scratchpad and when debugging an app in the app manager.  Is there a different way to reproduce?

Comment 4

[Panos Astithas (he/him) [:past] (please ni?)]

Hrmph, trying to reproduce on today's fx-team tip on Linux I can't get either Ctrl-I or ESC to have any effect. I'll try tomorrow on Mac where I first observed the problem and report back.

Comment 5

[Panos Astithas (he/him) [:past] (please ni?)]

OK, it looks like keybindings in the App Manager are broken at the moment. Here is another set of steps to reproduce this bug:

- open firefox in one profile and execute 'listen 6666' in gcli
- open firefox in a second profile and use the 'Connect...' menu item to connect remotely to localhost:6666
- select one of the open tabs to debug
- go to the scratchpad tab and type 'window', then hit Cmd-I, then ESC

You should be seeing a split console after the object inspector is dismissed.

Comment 6

[Brian Grinstead [:bgrins]]

(In reply to Panos Astithas [:past] from comment #5)
> OK, it looks like keybindings in the App Manager are broken at the moment.
> Here is another set of steps to reproduce this bug:
> 
> - open firefox in one profile and execute 'listen 6666' in gcli
> - open firefox in a second profile and use the 'Connect...' menu item to
> connect remotely to localhost:6666
> - select one of the open tabs to debug
> - go to the scratchpad tab and type 'window', then hit Cmd-I, then ESC
> 
> You should be seeing a split console after the object inspector is dismissed.

The behavior seems a bit different now.

Pressing window<cmd+I> opens the variables view, but pressing escape doesn't close it.  I notice this also in a normal scratchpad.

Comment 7

[Brian Grinstead [:bgrins]]

Attachment: escape-scratchpad-WIP.patch

This prevents split console from opening when esc is pressed in the filter textbox.  It's still an issue if another part of the variables view is focused, and still has the problem where pressing esc in the scratchpad doesn't close the view.

Comment 8

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

Scratchpad triage, filter on "ghostbusterzzz"

Comment 9

[Nicolas Chevobbe [:nchevobbe]]

Scratchpad was removed in Bug 1519103 (Firefox 72). You can check the alternative in https://wiki.developer.mozilla.org/en-US/docs/Tools/Deprecated_tools#Scratchpad .

The Scratchpad bug list was reviewed and I think this bug can be safely closed.