91526 - people.yahoo.com is far too wide

Status: VERIFIED FIXED

(Core :: Layout: Tables, defect)

Description

[Blake Ross]

Today's comm. branch build

people.yahoo.com is way too wide; looks like a table issue.

Comment 1

[Mats Palmgren (inactive)]

Attachment: Testcase

Comment 2

[Mats Palmgren (inactive)]

Problem seems to be caused by a NOWRAP attribute.

Comment 3

[selmer (gone)]

I think karnaze is out of the office.  Can anyone say what's the right thing to
do here?  Is this really an evangelism bug?  Is this really a feature of NOWRAP?

Comment 4

[Marc Attinasi]

It is not a feature or NOWRAP. There is a bug here, probably specific to
processing of the NOWRAP attribute.

Comment 5

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

When did this bug start?

There's no spec for this type of thing, so it's hard to say if it's a bug with
the page.  What's happening is that they specify a table with 4 cells (the
upper-right one is not in the document and implied, and the lower two are
joined) where the upper left cell should have a width of 1% of the entire table.
 However, IE (and this is consistent with what CSS says about percentage widths)
ignores this percentage width, presumably since the parent has no fixed with and
the width attribute on TD acts more like a min-width.  Then we expand the parent
based on the child, which seems wrong.  Maybe it's happening because we end up
with four reflow passes due to the nested tables?

Karnaze should be around.  He was yesterday, at least.

Comment 6

[/\/\arcio Galli]

I will try to simplify this. 

Comment 7

[Alexandru Savulov]

reassigning to me since it seems to be realted to 80817

Comment 8

[Bob Clary [:bc] (inactive)]

If you change all the width="1%" to width="0" the page will display much better
although still not exactly like IE.

Comment 9

[/\/\arcio Galli]

Will add my testcase demonstrating the problem with width="1%". Agreed that the
NOWRAP is also another workaround to the problem. 

Comment 10

[/\/\arcio Galli]

Attachment: testcase with two tables. First one have cell with width="1%"

Comment 11

[Alexandru Savulov]

second test case (attachment 44169 [details]) gets handled like IE does - where is the
problem here?

Comment 12

[Alexandru Savulov]

fixed on trunk

Comment 13

[Amarendra Hanumanula]

Alexandru Savulov, second testcase attached works fine but the first test case 
attached by Mats Palmgren does not work and IE handles it differently. The 
NOWRAP seems to be a problem here. I am reopening this bug and if you think that 
the original problem described in this bug was fixed, you can close this bug

Comment 14

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Other issues should be filed as a separate bug.  Please describe actual vs.
expected results more clearly.  "IE handles it differently" doesn't help those
people who don't have IE and it doesn't say which version of IE you're talking
about.

Comment 15

[Alexandru Savulov]

Comment on attachment 43334 [details]
Testcase

opened bug 120191 for this testcase since it didn't displayed the problem of
the URL

Comment 16

[Alexandru Savulov]

closing the bug FIXED since URL works

opened bug 120191 for the first (obsoleted) testcase

Comment 17

[Amarendra Hanumanula]

 A new bug was opened for the first testcase failure. The second testcase works
fine. I am closing this bug. Verified.