Polish the box model properties

RESOLVED FIXED in Firefox 55

Status

enhancement
P3
normal
RESOLVED FIXED
2 years ago
Last year

People

(Reporter: gl, Assigned: lockhart)

Tracking

(Blocks 1 bug)

unspecified
Firefox 55
Dependency tree / graph

Firefox Tracking Flags

(firefox55 fixed)

Details

Attachments

(1 attachment)

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: 2 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.