Bug 5066 – Line breaks are allowed between images when they shouldn't be

• Component: Layout and Rendering
• Status: RESOLVED DUPLICATE

[Peter Larsen]

Not much to say exept that graphics clutter. It works perfectly fine in Opera 8.5.2173.

I have tested with latest nightly from 17th september and Safari 2.0.1 (412.5)

[Joost de Valk (AlthA)]

Confirmed in ToT. Moving to p2 since i think p1 is a big tough here.

[Eric Seidel (no email)]

Agreed.  Layout is wrong.  We need a reduced test case before we can do much with this bug however:
http://webkit.opendarwin.org/quality/reduction.html

[Joost de Valk (AlthA)]

Created attachment 5335 [details]
Testcase

for some reason, safari puts these images below eachother, instead of next to
each other.

[Joost de Valk (AlthA)]

I don't know if this is a bug, the table has width 10%, and safari is obeying that it seems. Changing 
keyword to HasReduction.

[Dave Hyatt]

This really has nothing to do with tables.  We allow line breaks between images that have no whitespace between them.  Firefox, WinIE, and Opera don't.  This is a file bug in Radar from years ago... not sure if it ever made it into Bugzilla.

[Dave Hyatt]

This behavior is actually completely unspecified in CSS unfortunately, but I agree we should match WinIE and Firefox.  Unfortunately their treatment of replaced elements is completely inconsistent (form controls behave differently than images, etc.), but I guess we have to match.

[Dave Hyatt]

Need a giant matrix basically of every possible replaced element type next to every other possible type, e.g., image-image, select-image, image-select, select-select, button-image, image-button, etc.

Then after doing that, need to see what the behavior is when white-space:pre is put on one of the elements (on only the one before, on only the one after, etc.).

Note we have some form control-related issues already from the use of white-space:pre.

I think this bug was about the tables case, which was duplicated in bug 10517. The general behavior for replaced elements has already been addressed by a different patch.

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