Closed
Bug 79738
Opened 24 years ago
Closed 24 years ago
[PATCH] NAME attribute on MAP element is getting whitespace stripped out
Categories
(Core :: Layout, defect, P2)
Core
Layout
Tracking
()
VERIFIED
FIXED
mozilla0.9.1
People
(Reporter: attinasi, Assigned: attinasi)
References
Details
Attachments
(3 files)
(deleted),
text/html
|
Details | |
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
patch
|
Details | Diff | Splinter Review |
From Bugzilla Helper:
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; m18) Gecko/20010131
Netscape6/6.01
BuildID:
If a NAME attribute on a MAP element has a value with whitespace, it is changed
to not have any whitesapce. This is incorrect. It was originally implemented
that way as a NavQuirk, however since IE does not implement that quirk it is
probably not important and shoudl be removed.
Reproducible: Always
Steps to Reproduce:
Create a MAP element with name="one two" and query the name attribute value via
the DOM
Actual Results: value is "onetwo"
Expected Results: value is "one two", as it was originally set.
Assignee | ||
Comment 2•24 years ago
|
||
Assignee | ||
Comment 3•24 years ago
|
||
Assignee | ||
Comment 4•24 years ago
|
||
Updating milestone and priority...
Priority: -- → P2
Target Milestone: --- → mozilla0.9.1
Assignee | ||
Comment 5•24 years ago
|
||
Patch is attached: Harish, can you review please?
Keywords: patch
Summary: NAME attribute on MAP element is getting whitespace stripped out → [PATCH] NAME attribute on MAP element is getting whitespace stripped out
Comment 7•24 years ago
|
||
This morning's builds had a bug ( bug 80746 ) which may have led to
a Bugzilla user inadvertantly
changing this bug from the Assignbed/Accepted status to the New status. If you
are the owner of this bug please check to see that it is in the correct Status.
Thanks.
Comment 8•24 years ago
|
||
It looks that we need sr= here. Good progress.
First, please refer to the bugzilla bug, not the private bugscape one, in your
comment. Second, what about just tearing that code out, instead of ``#if 0''ing
it? Perhaps you could just put a comment to the effect that:
/*
* We used to strip whitespace from the NAME attribute, to match a 4.x quirk, but
* it proved too quirky for us, and IE never did that. See bug 79738 for details.
*/
Assignee | ||
Comment 11•24 years ago
|
||
shaver, absolutely - I'll do it. Nice wording, btw - I'll use it directly.
Assignee | ||
Comment 12•24 years ago
|
||
sr=shaver, in spite of your non-unified diff! =)
Assignee | ||
Comment 14•24 years ago
|
||
Some people cannot be pleased ;)
Assignee | ||
Comment 15•24 years ago
|
||
Fix checked in.
nsHTMLContentSink.cpp 3.447
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Comment 16•24 years ago
|
||
Verified on:
build: 2001-05-21-11-Mtrunk
platform: WinNT
The value of the name attribute is "one two".
Status: RESOLVED → VERIFIED
SPAM. HTML Element component deprecated, changing component to Layout. See bug
88132 for details.
Component: HTML Element → Layout
You need to log in
before you can comment on or make changes to this bug.
Description
•