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)

defect

Tracking

()

VERIFIED FIXED
mozilla0.9.1

People

(Reporter: attinasi, Assigned: attinasi)

References

Details

Attachments

(3 files)

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.
Accepting
Status: NEW → ASSIGNED
Attached file Testcase showing bug (deleted) —
Attached patch Proposed Patch (deleted) — Splinter Review
Updating milestone and priority...
Priority: -- → P2
Target Milestone: --- → mozilla0.9.1
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
r=harishd
Status: ASSIGNED → NEW
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.
It looks that we need sr= here. Good progress.
back to assigned...
Status: NEW → ASSIGNED
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. */
shaver, absolutely - I'll do it. Nice wording, btw - I'll use it directly.
sr=shaver, in spite of your non-unified diff! =)
Some people cannot be pleased ;)
Fix checked in. nsHTMLContentSink.cpp 3.447
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Blocks: 75664
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.

Attachment

General

Creator:
Created:
Updated:
Size: