Closed Bug 1457804 Opened 4 years ago Closed 4 years ago

Grid overlay is displayed in the wrong position for direction:rtl grid containers

Categories

(DevTools :: Inspector, defect, P2)

Product:
DevTools
Component:
Inspector
Type:
defect

Tracking

(firefox59 wontfix, firefox60 wontfix, firefox61 fixed)

Status:
RESOLVED FIXED
Milestone:
Firefox 61
Tracking Flags:
Tracking Status
firefox59 --- wontfix
firefox60 --- wontfix
firefox61 --- fixed

People

(Reporter: mats, Assigned: jryans)

References

(Blocks 1 open bug)

Details

Attachments

(4 files)

Testcase
1.21 KB, text/html
Details
Screenshot
9.05 KB, image/png
Details
Testcase #2
1.17 KB, text/html
Details
Bug 1457804 - Use content size for DevTools highlighter writing mode adjustments.
59 bytes, text/x-review-board-request
gl
: review+
Details
Attached file Testcase — 
STR:
1. load the attached testcase
2. open devtools inspector and select one of the grid elements
3. activate the grid overlay

ACTUAL RESULT
The grid overlay is rendered in the padding area.

ACTUAL RESULT
The grid overlay is rendered in the content area.
Attached image Screenshot — 
The light blue areas are the padding areas.
Errrh, replace "padding" with "border" above of course...
Attached file Testcase #2 — 
Actually, the bug occurs for padding too...
Flags: needinfo?(jryans)
Blocks: dt-grid
Component: Developer Tools → Developer Tools: Inspector
Assignee: nobody → jryans
Status: NEW → ASSIGNED
Flags: needinfo?(jryans)
Priority: -- → P2
Comment on attachment 8973078 [details]
Bug 1457804 - Use content size for DevTools highlighter writing mode adjustments.

https://reviewboard.mozilla.org/r/241618/#review247430
Attachment #8973078 - Flags: review?(gl) → review+
Pushed by jryans@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/daaa9249868c
Use content size for DevTools highlighter writing mode adjustments. r=gl
https://hg.mozilla.org/mozilla-central/rev/daaa9249868c
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
status-firefox61: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 61
QA Whiteboard: [good first verify]
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.