WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED CONFIGURATION CHANGED
56153
RenderBlocks with negative width
https://bugs.webkit.org/show_bug.cgi?id=56153
Summary
RenderBlocks with negative width
James Robinson
Reported
2011-03-10 17:07:15 PST
When loading the given URL a set of RenderBlocks have negative widths, which causes a number of problems. I'm currently bisecting to figure out when this started happening as it seems to be a fairly recent regression.
Attachments
Add attachment
proposed patch, testcase, etc.
James Robinson
Comment 1
2011-03-11 19:23:33 PST
Repro: <!DOCTYPE html> <div id="d" style="margin-left: 100%; position: absolute; left: 20px; right: 160px;"></div> d.clientWidth is -180 in WebKit and 0 in Firefox 3.6, Firefox 4 beta, Opera 10.62, and IE8. The -180 value seems to come from here:
http://trac.webkit.org/browser/trunk/Source/WebCore/rendering/RenderBox.cpp#L2422
This doesn't appear to be a recent regression.
James Robinson
Comment 2
2011-03-14 13:14:31 PDT
After chatting with Tab it seems the issue is that when calculating the logical width we need to handle the case where the width goes negative by dropping a constraint and re-solving (dropping into the !logicalLeftIsAuto && !logicalWidthIsAuto && !logicalRightIsAuto case would seem to do the trick here). It looks like what Gecko 2.0 does is ignore the specified value for 'right' and re-solving - the computed values for left/right/width are 20px/-20px/0px, respectively in this test case. The same issue exists for padding/borders.
Ahmad Saleem
Comment 3
2023-05-20 09:56:29 PDT
Test Case (based on
Comment 01
):
https://jsfiddle.net/w17kymqg/2/
^ Though tweaked to output clientWidth. *** Safari 16.5 *** clientHeight: 0px clientWidth: 0px *** Chrome Canary 115 & Firefox Nightly 115 *** clientHeight: 0px clientWidth: 0px ________ We don't get -180, which this bug was about. So I am marking this as "RESOLVED CONFIGURATION CHANGED".
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug