Closed Bug 192902 Opened 22 years ago Closed 15 years ago

Words wrap in nowrap cell when width=0 and next cell have width=100%

Categories

(Core :: Layout: Tables, defect, P3)

defect

Tracking

()

RESOLVED FIXED
Future

People

(Reporter: joakim_46, Assigned: bernd_mozilla)

References

()

Details

(Keywords: testcase)

Attachments

(2 files)

User-Agent: Mozilla/4.0 (compatible; MSIE 6.0; Windows NT 5.1) Build Identifier: 2003021008 As you can see, the Sony Ericsson T200 (top) is outlined. Reproducible: Always Steps to Reproduce: 1. Start Mozilla 2. Load http://www.telemoveis.com/catalog/itemcomments.asp?id=841 3. Actual Results: The page opens correctly, but the Sony Ericsson T200 shouldn't be outlined. Expected Results: The Sony Ericsson T200 text should be displayed correctly.
From the code: <td background="/images/sub_folder_mid.gif" width="0" nowrap="nowrap" class="titulosproduto">Sony Ericsson T200</td> The td has a width="0", though it also has nowrap.
tables; sounds like nowrap broke....
Assignee: asa → table
Component: Browser-General → Layout: Tables
QA Contact: asa → madhur
Though the problem seems to be something else, cause this works fine: <table> <tr> <td width="0" nowrap="nowrap">Does this wrap?</td> </tr> </table>
Priority: -- → P3
Target Milestone: --- → Future
-> All/All (OS X 1.4a)
OS: Windows XP → All
Hardware: PC → All
Attached file Testcase
Keywords: testcase
Summary: Text outlined → Words wrap in nowrap cell when width=0 and next cell have width=100%
the pixel width on the td wins over the nowrap attribute (http://lxr.mozilla.org/seamonkey/source/content/html/content/src/nsHTMLTableCellElement.cpp#506) this might be not correct :-)
The question here is if we need this quirk. IE and webkit do not have it, Opera does what we do.
Attached patch patchSplinter Review
glacial speed, but here we go
Assignee: layout.tables → bernd_mozilla
Status: NEW → ASSIGNED
Attachment #373561 - Flags: superreview?(bzbarsky)
Attachment #373561 - Flags: review?(bzbarsky)
Comment on attachment 373561 [details] [diff] [review] patch Even more glacial review... r+sr=bzbarsky
Attachment #373561 - Flags: superreview?(bzbarsky)
Attachment #373561 - Flags: superreview+
Attachment #373561 - Flags: review?(bzbarsky)
Attachment #373561 - Flags: review+
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Depends on: 527464
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: