Closed Bug 1535015 Opened 6 years ago Closed 6 years ago

No flex diagram for flexitem/-container when remote debugging

Categories

(DevTools :: Inspector, defect)

65 Branch
defect
Not set
normal

Tracking

(firefox68 fixed)

RESOLVED FIXED
Firefox 68
Tracking Status
firefox68 --- fixed

People

(Reporter: mbalfanz, Assigned: mbalfanz)

References

(Blocks 1 open bug)

Details

Attachments

(5 files, 1 obsolete file)

Attached image screenshot-1 (deleted) —

STR:

  • Take an android device and enable Firefox USB debugging
  • Visit css-tricks.com
  • Connect to the tab through about:debugging-new
  • Inspect div.logo and open the layout sidebar

ER:

  • see a chart like in screenshot-1

AR:

  • no chart is displayed, screenshot-2
Attached image screenshot-2 (deleted) —
Attached image screenshot-3 (deleted) —

Final size seems to be calculated wrong.

Attached image Screenshot_20190315-113334.png (deleted) —

The fact that the diagram is missing seems to be expected as the code bails out when the final size is 0.
So the problem is that the final size is 0, but really it shouldn't.

Here's a screenshot of the remote device itself shared by Martin, which clearly shows that the element is visible and does have a size.
Martin also said that the computed view showed a non-0 size.

Attached file Bug 1535015 - Add null-check for mainMaxSize; r=pbro (obsolete) (deleted) —
Attachment #9051253 - Attachment is obsolete: true
Assignee: nobody → mbalfanz
Attachment #9051256 - Attachment description: Bug 1535015 - Add null-check for mainMaxSize; r=pbro → Bug 1535015 - Add null-check for mainMaxSize; r=mtigley
Status: NEW → ASSIGNED
Pushed by mtigley@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/51df20c6ee92 Add null-check for mainMaxSize; r=mtigley
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 68
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: