Closed Bug 43204 Opened 24 years ago Closed 24 years ago

CSS Padding crops cell width

Categories

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

Product:
Core
Component:
Layout: Tables
Platform:
x86
Windows 98
Type:
defect

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla0.8

People

(Reporter: tony.gorman, Assigned: karnaze)

References

(
URL
)

Details

(Keywords: testcase, Whiteboard: fix-in-hand)

Attachments

(1 file)

patch to fix the bug
1.07 KB, patch
Details | Diff | Splinter Review 
Checkout the example.

CSS Padding-left and padding-right applied to content in a table cell seem to 
reduce the total width of the cell. The decrease seems to be directly 
proportional to the padding amounts.
*** Bug 43378 has been marked as a duplicate of this bug. ***
setting status to New
Status: UNCONFIRMED → NEW
Ever confirmed: true
Marc: rows, row groups, col groups, cols should have zero padding (zero margins 
also). In this case it looks like the row is getting a nonzero padding. The 
style system should probably deal with this to make it easier later on.
Assignee: karnaze → attinasi
Forget the last comment, since arbitrary elements can be maped to rows and vice 
versa. This needs to be handled in the reflow state or table code. Reassigning 
back to me.
Assignee: attinasi → karnaze

    
  
Status: NEW → ASSIGNED
Whiteboard: fix-in-hand
Target Milestone: --- → M18

    
  
Keywords: testcase

    
  
QA Contact: desale → chrisd

    
    

    
  
Test URL has now changed. This is more a prompt for bug owner to put the patch
into the tree :-)

URL: http://www.nomad-jedi.com/temp/cellBu... → http://www.concept6.co.uk/mozilla/cel...

    
    

    
  
karnaze, can you get this reviewed and maybe checked into the trunk if it's too
late for the branch?


    
    

    
  
Chris,

I just looked into this bug with a rotten patch. May be you can help the patch
submitter to get the patch into the tree.

Bernd

    
    

    
  
Marking mozilla0.8 (need to re-test the patch).
Target Milestone: M18 → mozilla0.8

    
    

    
  
The patch was checked in.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED

    
    

    
  
QA contact update
QA Contact: chrisd → amar

          You need to log in
          before you can comment on or make changes to this bug.
        






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: