Closed
Bug 55917
Opened 24 years ago
Closed 23 years ago
bad layout of image alt-text in table with (missing) floated images
Categories
(Core :: Layout, defect, P5)
Tracking
()
RESOLVED
WORKSFORME
Future
People
(Reporter: karnaze, Assigned: attinasi)
Details
(Keywords: testcase)
Attachments
(2 files)
The table regression test, bug26553.html, looks bad. When I remove the
align=right on the <img> it looks ok. This is a recent regression.
Reporter | ||
Comment 1•24 years ago
|
||
P2 bugs -> future
Status: NEW → ASSIGNED
Priority: P3 → P2
Target Milestone: --- → Future
Comment 3•24 years ago
|
||
Upon managerial request, adding the "testcase" keyword to 84 open layout bugs that
do not have the "testcase" keyword and yet have an attachement with the word
"test" in the description field. Apologies for any mistakes.
Keywords: testcase
Comment 5•24 years ago
|
||
I don't understand this bug report's description. Can someone please explain
what the problem is? Thanks.
Reporter | ||
Comment 6•24 years ago
|
||
It is a poorly submitted bug. The test case is from table regression test
bug26553 (mozilla/layout/html/tests/table) and the images are missing.
Reassigning to attinasi.
Assignee: buster → attinasi
Status: ASSIGNED → NEW
Target Milestone: Future → ---
Assignee | ||
Comment 7•24 years ago
|
||
Assignee | ||
Comment 8•24 years ago
|
||
I added another testcase with valid images, otherwise the same as the original
testcase. I see no problems, so it is either FIXED or was INVALID from the
start. Note that the middle cell width is overspecified, and even given that,
the table is formatted correctly. Sanity checks welcome - marking INVALID -
reopen if I am missong something please.
Status: NEW → RESOLVED
Closed: 24 years ago
Resolution: --- → INVALID
Comment 9•24 years ago
|
||
Well, bug #71974 has been marked as a DUP of this one, but certainly is still
not fixed. So, I am going to reopen it.
Assignee | ||
Comment 10•24 years ago
|
||
Yup - bug 71974 is certainly a problem - I'm working on reducing that testcase now.
Comment 11•24 years ago
|
||
Hmm, should "foo" and "bar" really be on separate lines (1st testcase)? In the
2nd testcase they are on the same line... (and while loading 1st testcase you
can briefly see that it first places the images on the same line before it
decides to give up on those URLs).
Assignee | ||
Comment 12•24 years ago
|
||
I think you are right Mats - the foo and bar alt-text for the images should be
on the same line. I'll reopen to investigate some more. Updating summary to
reflect the problem more specifically.
Status: RESOLVED → REOPENED
Resolution: INVALID → ---
Summary: bad layout in table with align=right <img> → bad layout of image alt-text in table with (missing) floated images
Comment 13•23 years ago
|
||
wait a minute. looking at the code of the testcase, it seems that we are doing
the right thing. The table is set to width=100% -- which is fine, however, the
middle column is set to <td width="100%" ...> which would force the left and
right columns to be as small as possible. Since the alt text is from two
different images, it i=would be logical to get a return causing the second alt
text to go to the next line. Of course the empty anchor tag around the two
images does not help in the spacing either.
There are differences between IE and moz:
1. IE seems to ignore the 2nd column width of 100%
2. IE honors the specified width of the image, so the alt text is displayed
within the specified width and height
3. IE seems to adjust the width of the columns to fit the data, even though the
2nd column was set to 100
4.x seems to render similar to IE
This seems to be such a minor difference, I don't see why this is a P2, this
should be downgraded to P5/trivial
Updated•23 years ago
|
Severity: normal → trivial
Priority: P2 → P5
Target Milestone: --- → Future
Comment 14•23 years ago
|
||
removing myself from the cc list
Comment 15•23 years ago
|
||
WORKSFORME, build 2002-04-05-03 (trunk) on Windows 98 SE.
Status: REOPENED → RESOLVED
Closed: 24 years ago → 23 years ago
Resolution: --- → WORKSFORME
You need to log in
before you can comment on or make changes to this bug.
Description
•