Bug 50135 – Wrong positioning due to wrong width calculation wrt width:0

• Status: RESOLVED
• Resolution: FIXED
• Component: CSS

[crisp]

See testcase (also attached); when a blocklevel element is positioned to the right, containing another blocklevel element that has it's width set to 0 but which has a positive padding-left-value, it appears that the zero-width is ignored and the positioning of the containing element is wrong.

Changing the width to a non-zero value doesn't exhibit this behaviour.

[crisp]

Created attachment 74981 [details]
Testcase

[Pravin D]

Created attachment 140539 [details]
Patch

[Pravin D]

The issue is this:
When a block parent has a child element with width:0 specified, the width of the parent block is calculated as though the child element's width is not specified and takes the child elements width to set as auto. However the content of the child element will not be painted as the child knows its width is zero. 
The cause is that while checking for width type (fixed, auto , etc), in the case of type fixed , the width value is expected to be positive. The check however does not consider the case when width is zero.

[Eric Seidel (no email)]

Comment on attachment 140539 [details]
Patch

This looks reasonable.

[Pravin D]

(In reply to comment #4)
> (From update of attachment 140539 [details])
> This looks reasonable.

Thnk you

[Eric Seidel (no email)]

Comment on attachment 140539 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=140539&action=review

> Source/WebCore/ChangeLog:1
> [phone]  Pravin D  <[e-mail]>

Normally contributors write their full names here.

[Pravin D]

(In reply to comment #6)
> (From update of attachment 140539 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=140539&action=review
> 
> > Source/WebCore/ChangeLog:1
> > [phone]  Pravin D  <[e-mail]>
> 
> Normally contributors write their full names here.

Pravin D is my full name  or atleast how it is in official records :)

[Eric Seidel (no email)]

A pleasure to meet you then. :)

[Pravin D]

(In reply to comment #8)
> A pleasure to meet you then. :)

The pleasure mine. :)

[WebKit Review Bot]

Comment on attachment 140539 [details]
Patch

Clearing flags on attachment: 140539

Committed r116331: <http://trac.webkit.org/changeset/116331>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.