332551 - floated images with intrinsic dimensions and percentage based margins are positioned/painted incorrectly

Status: RESOLVED FIXED

(Core :: Layout: Floats, defect)

Description

[philippe (part-time)]

User-Agent:       Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.9a1) Gecko/20060402 Camino/1.2+
Build Identifier: Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.9a1) Gecko/20060402 Camino/1.2+

An image is floated and no dimensions (width/height) are specified, relying on the intrinsic size.
Percentage value is specified for the left-margin.
DomInspector reports the correct (computed) value for the margin, but the floated image is incorrectly positioned, too the left of where it should be.

Reproducible: Always

Actual Results:  
The image is positioned too much to the left.

Expected Results:  
Position the image as specified by the margin settings in the stylesheet.

Comment 1

[philippe (part-time)]

This can be reproduced with Firefox 1.5 (Win, Mac, Linux), Camino (Trunk, Gecko1.9a1), Mozilla 1.7 (Gecko 1.7)

Comment 2

[philippe (part-time)]

Attachment: test case

In the box on the second row, the image is positioned incorrectly

Comment 3

[philippe (part-time)]

Attachment: Screen shot

Comment 4

[Erik Fabert]

*** This bug has been marked as a duplicate of 249982 ***

Comment 5

[Boris Zbarsky [:bzbarsky]]

I'm not convinced that this is a duplicate, since bug 249982 is all about the float being a block, and in this case it's not...  Probably dependent, though.

Comment 6

[philippe (part-time)]

(In reply to comment #5)
> I'm not convinced that this is a duplicate, since bug 249982 is all about the
> float being a block, and in this case it's not...  Probably dependent, though.

Explicitly setting the image to display:block doesn't change the problem. I'll attach a test case next. So, pretty similar anyway.

PS - could you set bug 249982 to All/All (makes it easier to find, maybe :))

Comment 7

[philippe (part-time)]

Attachment: test case 2

Same test case, with the image set to img {display:block;}

Comment 8

[philippe (part-time)]

This fails : 20061207 build
Works: 20061208 build
--> fixed by bug 300030