15499 - Percentage value for cellpadding not working

Status: RESOLVED INVALID

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

Description

[Chris Petersen]

Build: Apprunner
Version: 1999100120
Platform: All
Expect Results: The cellpadding for each cell should be 50%.
What I got: No padding is applied

Steps to Reproduce:

1) Open test case:http://slip/projects/marvin/html/table_cellpadding_pct.html
2) The table cells are not rendered with a cellpadding of 50 %.

Comment 1

[karnaze (gone)]

adding CSS 2 to summary.

Comment 2

[Hixie (not reading bugmail)]

petersen: could you attach the test case to the bug? Cheers.

Comment 3

[Christine Hoffman]

Petersen is not in today - I have attached the test case below

Comment 4

[Christine Hoffman]

Attachment: test case to demonstrate problem

Comment 5

[Christine Hoffman]

In reviewing the test case, perhaps this is only a Nav Quirks bug. In the test
case provided, there is no height value, so I'm not sure what the 50% would
apply to. However, if you bring the test case up in Nav, the rendering is
different. Karanaze, can you shed some light on this?

Comment 6

[Christine Hoffman]

The attachment below is somewhat revised, setting a width and height to the
table. Padding seems to work on top and bottom, not left or right

Comment 7

[Christine Hoffman]

Attachment: revised test case

Comment 8

[troy]

*** Bug 15501 has been marked as a duplicate of this bug. ***

Comment 9

[Hixie (not reading bugmail)]

Migrating from {css2} to css2 keyword. The {css1}, {css2}, {css3} and {css-moz}
radars should now be considered deprecated in favour of keywords.
I am *really* sorry about the spam...

Comment 10

[Gervase Markham [:gerv]]

No change to the bug except added [TESTCASE] to Status line to get it off the
BugAThon list.

Gerv

Comment 11

[ekrock's old account (dead)]

Bulk moving [testcase] code to new testcase keyword. Sorry for the spam!

Comment 12

[Hugh Kennedy]

*** Bug 36061 has been marked as a duplicate of this bug. ***

Comment 13

[karnaze (gone)]

Moving to M18.

Comment 14

[karnaze (gone)]

This bug has been marked "future" because we have determined that it is not 
critical for netscape 6.0. If you feel this is an error, or if it blocks your 
work in some way -- please attach your concern to the bug for reconsideration.

Comment 15

[Hixie (not reading bugmail)]

*** Bug 50075 has been marked as a duplicate of this bug. ***

Comment 16

[Hixie (not reading bugmail)]

*** Bug 2779 has been marked as a duplicate of this bug. ***

Comment 17

[Christine Hoffman]

Percentage value for cellspacing also does not work. Should that be a separate 
bug?

Comment 18

[Amarendra Hanumanula]

QA contact update

Comment 19

[Mario Gaucher]

when I go to www.moncourrier.com (mozilla/mozilla for login/password test)
compose box when I press the "Ecrire" button on the left is not the correct
width... it should be as large as the grey bar just on the top.
It is ok with Internet explorer 5.5 or netscape 4.75 on Windows 2000

is it the same bug... or should I submit a new one?

Comment 20

[Amarendra Hanumanula]

Adding "nsbranch" keyword because % value for "cellpadding" and "cellspacing" 
are commonly used and important...

Comment 21

[Kevin McCluskey (gone)]

Marking nsbranch-

Comment 22

[Kevin McCluskey (gone)]

No fix in hand. Not a "stop ship" bug.

Comment 23

[anthonyd]

this looks like it works correctly.
anthonyd

Comment 24

[karnaze (gone)]

We don't yet support percent cellpadding and currently (as of a few days ago) 
turn percent values into pixel values like IE and Nav. So, if this bug is 
complaining that we are not compatable with IE then it is fixed. 

Comment 25

[kirill]

Attachment: Padding-left causing content to overlap right cell border

Comment 26

[kirill]

According to comment #24 the percent values should be converted to pixels. That 
does not seem to happen except for the padding-left where it is causing text to 
overlap the right border.

Comment 27

[karnaze (gone)]

mass reassign to default owner

Comment 28

[Boris Zbarsky [:bzbarsky]]

Frankly, I'm not sure we want to fix this bug.  Neither NS4 nor IE implement
this, and percent values for padding set via CSS work, so....

Comment 29

[Bernd]

Attachment: testcase

percent padding works ???

Comment 30

[Boris Zbarsky [:bzbarsky]]

Well, the cell has 10% padding on all sides.  This does cause it to overflow the
table, etc (but what exactly is the expected rendering there?  The table size
depends on the cell size, the cell size depends on the table size; unlike the
div, the available space for the cell is the table width, not the body width).

Comment 31

[Jan]

There is a problem with XHTML files.

Build identifier: Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.9a4pre) Gecko/2007040204 Minefield/3.0a4pre

See attachment below.

Comment 32

[Jan]

Attachment: Test page XHTML

Comment 33

[Boris Zbarsky [:bzbarsky]]

That testcase has nothing to do with this bug.  This bug is about cellpadding, not cellspacing.

Comment 34

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

HTML5 defines cellpadding to contain just pixel values. Marking invalid.

Comment 35

[Pulsebot]

Pushed by ffxbld-merge:
https://hg.mozilla.org/releases/mozilla-beta/rev/d1adc8a073f9
[fenix] For https://github.com/mozilla-mobile/fenix/issues/15499 - Smoothly updates topSites list on remove