Closed Bug 1350499 Opened 7 years ago Closed 7 years ago
Different width/height shown in editbox of boxmodel-content
66.85 KB, image/png
59 bytes, text/x-review-board-request
Steps to Reproduce: 1. Start latest Nightly (e.g. Build ID:20170324030205) 2. Go to about:home 3. Open Inspector > Computed 4. Select #aboutMozilla in markup 5. Click width/height on boxmodel-content Actual Result: Different width/height is shown. (It seems #aboutMozilla::before size?) Please refer to attached image. Expected Result: Is this intentionally? If not, own width/height is shown as default when editing.
We actually have the same issue with margin/border/padding etc... If a pseudo-element defines a value for any of those properties, and the element itself doesn't, the editing session will pick up the value from the pseudo element.
Assignee: nobody → jdescottes
Status: NEW → ASSIGNED
Comment on attachment 8852475 [details] Bug 1350499 - exclude styles for pseudo-els when building boxmodel widget; https://reviewboard.mozilla.org/r/124680/#review127246 ::: devtools/client/inspector/boxmodel/test/browser_boxmodel_editablemodel_pseudo.js:10 (Diff revision 1) > +"use strict"; > + > +// Test that pseudo elements have no side effect on the box model widget for their > +// container. See bug 1350499. > + > +const TEST_URI = "<style>" + We should just use multiline strings here
Attachment #8852475 - Flags: review?(gl) → review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/1f74f77a9814 exclude styles for pseudo-els when building boxmodel widget;r=gl
I have verified this bug was fixed in latest Nightly build (20170410030221). Thanks!
Status: RESOLVED → VERIFIED
[bugday-20170412] The bug is no longer to be reproducible in Latest Nightly 55.0a1. OS:Windows 10
You need to log in before you can comment on or make changes to this bug.