Closed
Bug 1478481
Opened 6 years ago
Closed 6 years ago
Flexbox highlighter overlay is wrong size, in page with full-page-zoom applied
Categories
(DevTools :: Inspector, defect, P3)
DevTools
Inspector
Tracking
(firefox64 verified)
VERIFIED
FIXED
Firefox 64
Tracking | Status | |
---|---|---|
firefox64 | --- | verified |
People
(Reporter: dholbert, Assigned: miker)
References
(Blocks 1 open bug)
Details
(Whiteboard: [dt-q])
Attachments
(2 files)
STR:
0. enable about:config prefs:
devtools.inspector.flexboxHighlighter.enabled
devtools.flexboxinspector.enabled
1. View some page with a flex container, e.g.:
data:text/html,<div style="display:flex;width:100px;height:100px; border: 1px solid black"><span>InspectMe
2. Zoom in with Ctrl+ as much as possible (or as much as you like)
3. Right-click the flex container and choose "Inspect", and click the flex badge in the inspector to activate the overlay.
ACTUAL RESULTS: The overlay boxes are much bigger than the boxes that they correspond to. It looks like the zoom factor has been applied twice to the overlay, or something like that.
EXPECTED RESULTS: Overlay should accurately highlight the area of the flex container & its flex item.
It looks like we get this correct for the grid overlay (e.g. if I change the data URL above to use "grid" instead of "flex" and activate that overlay). So whatever's broken here must be in some sizing piece that is different between the two.
I'm using Nightly 63.0a1 (2018-07-25) (64-bit)
Reporter | ||
Updated•6 years ago
|
Component: Computed Styles Inspector → Inspector
Reporter | ||
Updated•6 years ago
|
Priority: -- → P3
Reporter | ||
Comment 1•6 years ago
|
||
Updated•6 years ago
|
Blocks: dt-flexbox
Assignee | ||
Updated•6 years ago
|
Assignee: nobody → mratcliffe
Status: NEW → ASSIGNED
Assignee | ||
Comment 2•6 years ago
|
||
I keep getting pulled away from this to fix telemetry issues.
My original thought was simply to multiply by a zoom factor throughout the flexbox code but after doing that and then trying to tweek the rest it became clear that this is the wrong approach.
I am now going through the flexbox.js methods changing one line at a time, which has the advantage that I become much more familiar with the code.
I need to fix bug 1488835 then I will get back to this.
Assignee | ||
Updated•6 years ago
|
Whiteboard: dt-q
Updated•6 years ago
|
Whiteboard: dt-q → [dt-q]
Assignee | ||
Comment 3•6 years ago
|
||
[WIP] Need to find out why scrolling does not match up.
Assignee | ||
Updated•6 years ago
|
Attachment #9013997 -
Flags: review?(gl)
Comment 4•6 years ago
|
||
Can you describe the scrolling issue and why your fix will resolve it? I mostly want to make sure we can somehow document the issue and the fix. It might even be better if we simply spin up a new bug for the scroll issue since this patch really solves like 3 different things. I would also like to see the justify-content fix either in a separate patch or bug including a description of the problem, STR, etc since it's kinda magical right now.
Ideally, I would like to see this patch split into 3 different patches or bugs - one for addressing each issue.
Updated•6 years ago
|
Attachment #9013997 -
Flags: review?(gl)
Assignee | ||
Updated•6 years ago
|
Attachment #9013997 -
Flags: review?(gl)
Updated•6 years ago
|
Attachment #9013997 -
Flags: review?(gl)
Pushed by mratcliffe@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/7a275e7abf9b
Flexbox highlighter overlay is wrong size, in page with full-page-zoom applied r=gl
Comment 6•6 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
status-firefox64:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 64
Comment 7•6 years ago
|
||
Confirmed issue with 63.0a1 (2018-07-25) on Windows 10.
Verified with 64.0b9 on Win10, macOS10.13, Ubuntu16.04
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•