Closed Bug 104980 Opened 24 years ago Closed 24 years ago

oldschoolskateboards.net The imagemap uses the invalid form |usemap="#foo bar"| spaces are not allowed in uris.

Categories

(Tech Evangelism Graveyard :: English US, defect)

x86
Windows 2000
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WORKSFORME

People

(Reporter: wwsmurf, Assigned: doronr)

References

()

Details

(Whiteboard: aok)

possible breakage -- although the ncsa fishtank imagemap does work.
*** Bug 104981 has been marked as a duplicate of this bug. ***
.
Assignee: asa → bclary
Component: Browser-General → English: US
Product: Browser → Tech Evangelism
QA Contact: doronr → zach
Summary: The imagemap dosen't work??? -- did with 0.9.5 → oldschoolskateboards.net The imagemap uses the invalid form |usemap="#foo bar"| spaces are not allowed in uris.
Version: other → unspecified
However, after using the source(tm), I find that there is probably an obvious work around. Looking in nsImageMap.cpp, if this is the correct place, around line 134, we see aTarget (which I presume is the target in the usemap tag,) which is told to truncate, that is, remove the initial #. To me it appears that the old? nsImageMapUtils.cpp had a utility to fix this. It used the .StripWhiteSpace method of nsAReadableString to fix it "for navigator compatibility. Does it inherit this ability from nsString? If so, would inserting aTarget.StripWhiteSpace before or after line 134 in nsImageMap.cpp fix this problem?
Dupe of bug 93538?
Confirmed marking NEW. This bug is open only for evangelism issues since bug 87050 is fixed, but i'm not duping this for the reasons bclary gave in bug 93538.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Whiteboard: aok
Mass reassign of all tech-evangelism us general bugs assigned to bc to doron except bc's P1 bugs. You may search for this mass reassign (it is 305 bugs) by searching for the keyword 'greeneggsandham'
Assignee: bclary → doronr
don't see a problem
Status: NEW → RESOLVED
Closed: 24 years ago
Resolution: --- → WORKSFORME
Product: Tech Evangelism → Tech Evangelism Graveyard
You need to log in before you can comment on or make changes to this bug.