Closed Bug 2119 Opened 26 years ago Closed 24 years ago

{compat} minor overlap of floated table and text

Categories

(Core :: Layout, defect, P2)

Product:
Core
Component:
Layout
Platform:
x86
Windows 95
Type:
defect

Tracking

()

Status:
VERIFIED FIXED

People

(Reporter: mnalis, Assigned: buster)

References

(
URL
)

Details

Attachments

(1 file)

simplified test case
632 bytes, text/html
Details 
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 — 

    
    

    
  
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 ago → 24 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: