% cellspacing, cellpadding values should be treated as pixel values in quirks mode

VERIFIED FIXED in mozilla0.9.6

Status

()

Core
Layout: Tables
P1
normal
VERIFIED FIXED
16 years ago
16 years ago

People

(Reporter: karnaze (gone), Assigned: karnaze (gone))

Tracking

({testcase})

Trunk
mozilla0.9.6
x86
Windows 2000
testcase
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: PATCH CANDIDATE_094)

Attachments

(2 attachments)

(Assignee)

Description

16 years ago
For compatibility reasons, % cellpadding, cellspacing values need to be treated 
as pixel values.

<table>
 <tr>
  <td>foo</td>
  <td>bar</td>
 </tr
</table>
(Assignee)

Comment 1

16 years ago
add cellspacing=40% cellpadding=20%
Status: NEW → ASSIGNED
Keywords: testcase
Target Milestone: --- → mozilla0.9.6
(Assignee)

Comment 2

16 years ago
Created attachment 54941 [details]
testcase
(Assignee)

Comment 3

16 years ago
Created attachment 54942 [details] [diff] [review]
patch to fix the bug
(Assignee)

Updated

16 years ago
Priority: -- → P1
Whiteboard: PATCH

Comment 4

16 years ago
Comment on attachment 54942 [details] [diff] [review]
patch to fix the bug

sr=attinasi
Attachment #54942 - Flags: superreview+

Comment 5

16 years ago
Comment on attachment 54942 [details] [diff] [review]
patch to fix the bug

r= alexsavulov
Attachment #54942 - Flags: review+
(Assignee)

Comment 6

16 years ago
The patch is in.
Status: ASSIGNED → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED
(Assignee)

Comment 7

16 years ago
The patch turns % cellpadding values in standard mode into pixel values as well, 
since we don't correctly support % cellpadding yet.
(Assignee)

Updated

16 years ago
Whiteboard: PATCH → PATCH CANDIDATE_094

Comment 8

16 years ago
The patch did fix the bug.. Marking verifed
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.