Closed Bug 2119 Opened 26 years ago Closed 25 years ago

{compat} minor overlap of floated table and text

Categories

(Core :: Layout, defect, P2)

x86
Windows 95
defect

Tracking

()

VERIFIED FIXED

People

(Reporter: mnalis, Assigned: buster)

References

()

Details

Attachments

(1 file)

See the url above. At the #feb anchor, there is table called 'Linux accordint to Jesse Berst'. It is supposed to be left aligned, using 40% of width. It does, but the text that should be on the right of the table (and is in other browsers, as for example netscape 4.08). Using nightly build of mozilla 990101 (and 981229), it puts that text over the table producing unreadable mish-mash. Also, anchors do not work (as in URL above. It goes to to beggining of the page, not at specified anchor #feb)
Assignee: toshok → rickg
Reassigning from toshok to rickg
Assignee: rickg → buster
Steve -- this looks like a table bug. (Yes I know you're trying to dump tables onto Chris.) Can you please verify this and reassign to chris if you believe it's a table problem?
Status: NEW → ASSIGNED
Setting all current Open/Normal to M4.
Product: MozillaClassic → Browser
per leger, assigning QA contacts to all open bugs without QA contacts according to list at http://bugzilla.mozilla.org/describecomponents.cgi?product=Browser
changed misc bugs to M6
setting my table bugs to M9
Assignee: buster → karnaze
Status: ASSIGNED → NEW
Chris: somehow, a couple of table bugs slipped through my filter. reassigning to you.
Assignee: karnaze → kipp
Target Milestone: M9 → M10
This overlappng problem is gone, but there remains a minor spacing problem between the floating table and the text to its right. Nav4.6 and IE5 put some space between the table and the text, Gecko does not. Here is a simple test. Reassigning to Kipp. <html><body> <p> <table width="40%" align="left" border> <tr> <th>Linux according to Jesse Berst</th></tr> <tr> <td> <font color=#336666>"I think it's great if you are willing to promote Linux to your boss. As long as you are aware of the risk you are taking. The risk of getting fired."</font></td></tr> </table> <b>Eric Raymond and friends come up with the "open source" term</b>. They apply for trademark status, and put up the <a href="http://www.opensource.org">opensource.org</a> web site. Thus begins the formal effort to push Linux for corporate use. <body></html>
Severity: normal → major
Status: NEW → ASSIGNED
Priority: P2 → P5
Summary: tables layed out incorrectly, overlapping non-table text → {compat} tables layed out incorrectly, overlapping non-table text
Target Milestone: M10 → M11
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
The final issue (not enough spacing around the floated table) has been fixed. I updated ua.css to have the necessary rules.
Status: RESOLVED → VERIFIED
With the Sept 2nd build, the problem has been fixed.
Attached file simplified test case (deleted) —
Re-opened for the "minor spacing problem" Chris describes below, because the fix for bug #25617 required removing the four rules from html.css that were added to give a small margin to floated tables and images. That approach won't work, because we're making the floated element wider than it should be Here are the four lines I removed from html.css: Index: html.css =================================================================== RCS file: /cvsroot/mozilla/layout/html/document/src/html.css,v retrieving revision 3.45 diff -r3.45 html.css 154,159d153 < table[align=left] { < margin-right: 4px; < } < table[align=right] { < margin-left: 4px; < } 419,425d412 < } < /* note: not exactly 4.x compatible; 4.x has 3px for *both* margins */ < img[align=left] { < margin-right: 4px; < } < img[align=right] { < margin-left: 4px;
Status: VERIFIED → REOPENED
Resolution: FIXED → ---
Status: REOPENED → ASSIGNED
Summary: {compat} tables layed out incorrectly, overlapping non-table text → {compat} minor overlap of floated table and text
Priority: P5 → P2
Target Milestone: M11 → M15
mine! mine mine mine! all mine! whoo-hoo!
Assignee: kipp → buster
Status: ASSIGNED → NEW
fixed
Status: NEW → RESOLVED
Closed: 25 years ago25 years ago
Resolution: --- → FIXED
Fixed in the April 21 build.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: