If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

[debugger.html] The new debugger is not easy to use when the devtools are docked to the right

RESOLVED FIXED

Status

()

Firefox
Developer Tools: Debugger
P2
normal
RESOLVED FIXED
8 months ago
7 months ago

People

(Reporter: julienw, Unassigned)

Tracking

({regression})

unspecified
regression
Points:
---

Firefox Tracking Flags

(firefox51 unaffected, firefox52 disabled, firefox53 disabled, firefox54 fixed)

Details

Attachments

(3 attachments)

(Reporter)

Description

8 months ago
STR:
1. Open the debugger devtools
2. Click on "dock to side of browser window".

Expected:
* the layout adjusts so that the sources and assets are at the bottom instead of the left.

Actual:
* the assets window stays at the left of the devtools window, which makes it very difficult to use the debugger with this layout.

Note this is a regression from the current debugger that properly adjusts its layout in this case.
(Reporter)

Comment 1

8 months ago
Created attachment 8831625 [details]
new debugger
(Reporter)

Comment 2

8 months ago
Created attachment 8831626 [details]
old debugger
(Reporter)

Updated

8 months ago
Keywords: regression
(Reporter)

Updated

8 months ago
status-firefox53: --- → affected
status-firefox54: --- → affected
(Reporter)

Updated

8 months ago
Summary: [debugger.html] The new debugger is not easy to use when the devtools are dicked to the right → [debugger.html] The new debugger is not easy to use when the devtools are docked to the right

Updated

8 months ago
status-firefox51: --- → unaffected
status-firefox52: --- → affected
The new debugger isn't riding the 52 train.
status-firefox52: affected → disabled
Julien, can you try this out again in Nightly?

We've landed some fixes for the vertical layout.  Still some tweaks needed but it should be much better.  Thanks.
Flags: needinfo?(felash)
Priority: -- → P2
Created attachment 8839593 [details]
Screen Shot 2017-02-21 at 10.37.39 AM.png

Here's a screenshot
(Reporter)

Comment 6

7 months ago
Thanks, it's better. 

But I think it's not enabled soon enough. There is a range of widths where I think this behavior should be enabled but isn't. So you need to tweak a little bit the step position :)

But it looks good !
Flags: needinfo?(felash)
@julien: please follow up more on GitHub with updated ranges, happy to explore this more.
Status: NEW → RESOLVED
Last Resolved: 7 months ago
Resolution: --- → FIXED
Is there a patch somewhere that can be backported to 53?
status-firefox54: affected → fixed
Flags: needinfo?(jlaster)
(Reporter)

Comment 9

7 months ago
TBH given the current state of the new debugger I highly doubt we'll keep it for 53.
I confirmed with Jason that the new debugger isn't riding the 53 train, so this isn't an issue there.
status-firefox53: affected → disabled
Flags: needinfo?(jlaster)
You need to log in before you can comment on or make changes to this bug.