Show scrollable elements in the markup-view using markup-badges

RESOLVED FIXED in Firefox 66

Status

enhancement
P3
normal
RESOLVED FIXED
3 months ago
2 months ago

People

(Reporter: pbro, Assigned: pbro)

Tracking

(Blocks 1 bug)

unspecified
Firefox 66
Dependency tree / graph

Firefox Tracking Flags

(firefox66 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

3 months ago

Debugging unwanted scrollbars in a web page often starts with identifying what element is having the scrollbar.
This is sometimes more difficult than it seems as many elements may be nested into each other, sharing the same size.

The idea of this feature is to make use of the markup-badges we already use for flexbox, grid, or events, by adding one for scrollable elements.

(Assignee)

Updated

3 months ago
Assignee: nobody → pbrosset
Status: NEW → ASSIGNED
(Assignee)

Comment 1

3 months ago

Detect when nodes are scrollable on the actor side, and use this
information to display new [scrollable] markup-badges in the inspector.
Also track this dynimically so badges are added/removed when necessary.
Hide this behind a pref for now, until this gets tested by users and
validated.

Comment 3

3 months ago
Pushed by pbrosset@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/21cb7caef246
Add new scrollable markup badges; r=gl

Comment 4

3 months ago
bugherder
Status: ASSIGNED → RESOLVED
Last Resolved: 3 months ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 66
(Assignee)

Updated

3 months ago
Blocks: 1523543
(Assignee)

Updated

2 months ago
Blocks: 1529280
(Assignee)

Updated

2 months ago
Duplicate of this bug: 1342028
You need to log in before you can comment on or make changes to this bug.