When both left and right are supplied the width of the element stretches. Hover when the box sizing is not set to content-box the width is incorrect. This is most likely because the box sizing property is incorrectly ignored in this case.
Created attachment 137044 [details] A div with both left and right inside another div No red should be seen at any time
Created attachment 137066 [details] [diff] [review] Fix (diff -u) Actually, the problem is that the box-sizing property is _applied_ in this case (to widths/heights that already subtract off the paddings/borders in their computation). It should not be.
Comment on attachment 137067 [details] [diff] [review] Fix (diff -uw) David, would you review? The CSSParser changes were prompted by some of the (unjustified, without those changes, hence commented out) layout assertions... and happen to fix a CSS-compliance bug as far as I can tell.
14 years ago
Checked in for 1.7a.
Since this checkin, I am getting the following assertion several hundred times when running the editor: ###!!! ASSERTION: Negative Width Input - very bad: 'mComputedWidth >= 0', file /home/pkw/sb/mozilla/trunk/mozilla/layout/html/base/src/nsHTMLReflowState.cpp, line 2532 Break: at file /home/pkw/sb/mozilla/trunk/mozilla/layout/html/base/src/nsHTMLReflowState.cpp, line 2532
Philip, would you mind filing a separate bug on me on that assert? I'll have to debug to see what's up....
Filed Bug 230249 for the assertion.
Fix causes a new bug 232754.