The default bug view has changed. See this FAQ.

[FIX] A map definition inside a table and outside the tr element will add an extra space to the table

RESOLVED FIXED

Status

()

Core
HTML: Parser
RESOLVED FIXED
15 years ago
3 years ago

People

(Reporter: Roger C. Soares, Assigned: mats)

Tracking

({compat, testcase})

Trunk
x86
All
compat, testcase
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments, 1 obsolete attachment)

(Reporter)

Description

15 years ago
A table will have an extra space if a map is defined inside the table (see URL -
the first frame).

For other elements, the element is moved to outside the table and the table is
rendered fine.
(Reporter)

Comment 1

15 years ago
Created attachment 85197 [details]
Testcase showing a map inside a table and a br inside a table

The map element will cause an extra space in the table, the br element will be
move to before the table.
To parser.
Assignee: karnaze → harishd
Status: UNCONFIRMED → NEW
Component: HTMLTables → Parser
Ever confirmed: true
QA Contact: amar → moied

Comment 3

15 years ago
Harish, 

We raised this one from the Evangelism Latin America effort. The site is part of
Major AOL partner in Brazil (Itau Bank). 

Comment 4

15 years ago
I see. If I use the <map> element the generated DOM is: 
-------------------------------------------->
<table bgcolor="green">
<map></map>
<tbody>...

When I replace to another element (For example a div element), I have:
-------------------------->
<div></div>
<table bgcolor="green">
<tbody>...

Harish, 

Do you think we can consider the MAP like the other elements?

Updated

15 years ago
Keywords: testcase
The URL has been redesigned to eliminate the problem.
(Assignee)

Comment 6

14 years ago
Created attachment 126244 [details]
Testcase #2
Attachment #85197 - Attachment is obsolete: true
(Assignee)

Comment 7

14 years ago
Taking this since I have a simple fix for it...
Assignee: harishd → mats.palmgren
Blocks: 7954
Summary: A map definition inside a table and outside the tr element will add an extra space to the table → [FIX] A map definition inside a table and outside the tr element will add an extra space to the table
Whiteboard: [patch]
(Assignee)

Comment 8

14 years ago
Created attachment 126246 [details] [diff] [review]
Patch rev. 1

Remove MAP as valid child of TABLE/THEAD/TFOOT
(Assignee)

Comment 9

14 years ago
The patch makes us compatible with NS4 and IE5.5/IE6.  This is also the right
thing to do with respect to HTML 4 spec.  Opera7 have *exactly* the same bug
as Mozilla currently has though.  I have tested this extensively; the regression
tests pass, the top100 list pass (which means none of them depend on this bug).
I have checked all Parser bugs that mentions "map" and "table" in the same bug
and found no regressions (among them I found bug 58942 which fixed the same bug
for TR).  I also checked all Layout bugs matching "<map" and "table" and saw
no problems.
Status: NEW → ASSIGNED
(Assignee)

Updated

14 years ago
Attachment #126246 - Flags: review?(harishd)

Updated

14 years ago
Whiteboard: [patch] → [patch] [needs review]

Comment 10

14 years ago
Comment on attachment 126246 [details] [diff] [review]
Patch rev. 1

r=harishd
Attachment #126246 - Flags: review?(harishd) → review+

Comment 11

13 years ago
Mats what is the status of this patch? Seems like it has a review :-)
(Assignee)

Updated

12 years ago
Attachment #126246 - Flags: superreview?(dbaron)
Mats, is there still interest in this patch?
Attachment #126246 - Flags: superreview?(dbaron) → superreview+
(Assignee)

Comment 13

12 years ago
Checked in 2005-01-25 16:40 PDT.

-> FIXED
Status: ASSIGNED → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → FIXED
Whiteboard: [patch] [needs review]
You need to log in before you can comment on or make changes to this bug.