Closed Bug 1349686 Opened 7 years ago Closed 7 years ago

Polish the box model properties

Categories

(DevTools :: Inspector, enhancement, P3)

enhancement

Tracking

(firefox55 fixed)

RESOLVED FIXED
Firefox 55
Tracking Status
firefox55 --- fixed

People

(Reporter: gl, Assigned: lockhart)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

Currently, the box model properties are styled exactly like the computed view, but given the space we have and the fact that it doesn't work like the computed view, we should make sure they don't overflow and fold into the next line.
Priority: -- → P3
Assignee: nobody → lockhart
Status: NEW → ASSIGNED
I think we should also change .boxmodel-position.boxmodel-top, .boxmodel-position.boxmodel-bottom to calc(50% - 2px) while we are at it. This seems more centered than - 1px.
Comment on attachment 8852249 [details]
Bug 1349686: Polish the Box Model CSS.

https://reviewboard.mozilla.org/r/124474/#review127342

Give my suggestions a try and let me know what you think.

::: devtools/client/themes/boxmodel.css:328
(Diff revision 1)
>  .boxmodel-properties-wrapper {
>    padding: 0 9px;
>  }
>  
> +.boxmodel-properties-wrapper .property-name-container {
> +  width: 50%;

I think we can just do "flex: 1" here.

::: devtools/client/themes/boxmodel.css:332
(Diff revision 1)
> +.boxmodel-properties-wrapper .property-name-container {
> +  width: 50%;
> +}
> +
> +.boxmodel-properties-wrapper .property-value-container {
> +  width: 50%;

And "flex: 1" here as well.
Attachment #8852249 - Flags: review?(gl)
Comment on attachment 8852249 [details]
Bug 1349686: Polish the Box Model CSS.

https://reviewboard.mozilla.org/r/124474/#review127474
Attachment #8852249 - Flags: review?(gl) → review+
https://hg.mozilla.org/mozilla-central/rev/8d2b523a1a71
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 55
I have successfully reproduced this issue on firefox nightly according to(2017-03-22)

Fixing bug is verified on Latest Nightly-- Build ID: (20170402030202),User Agent: Mozilla/5.0 (Windows NT 10.0; rv:55.0) Gecko/20100101 Firefox/55.0

Tested OS -- Windows10 32bit

[bugday-20170329]
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: