Closed
Bug 126982
Opened 23 years ago
Closed 23 years ago
imagemaps inactive if USEMAP attribute contains a filename fragment before the #
Categories
(Core :: Layout, defect)
Core
Layout
Tracking
()
People
(Reporter: glazou, Assigned: asa)
References
()
Details
Attachments
(1 file)
(deleted),
patch
|
fabian
:
review+
|
Details | Diff | Splinter Review |
1. launch Mozilla
2. open http://daniel.glazman.free.fr/testMAP.htm
3. the first imagemap is inactive because of its USEMAP attribute not beginning
by '#' or not reduced to a fragment identifyer
This bug blocks linguistic versions of http://www.sncf.com/, the french railways
company. I'll set a dependency to the TechEvangelism bug I filed.
Note : MacIE has same bug :-) :-) :-)
Reporter | ||
Comment 1•23 years ago
|
||
*** Bug 126973 has been marked as a duplicate of this bug. ***
Reporter | ||
Comment 2•23 years ago
|
||
Btw, code in nsContextMenu.js lines 226 to 228 is wrong :
if ( mapName ) {
// Find map.
var map = this.target.ownerDocument.getElementById( mapName.substr(1) );
this assumes
(a) that the <map> associated to the image and the image itself are
in same document
(b) that the USEMAP attribute always starts with a '#'
Reporter | ||
Comment 3•23 years ago
|
||
This patch solves the bug. It does not contain a fix for the context menu bug
I detected. Will submit another fix about the context menu asap.
Reviews please ?
Comment 4•23 years ago
|
||
Comment on attachment 70719 [details] [diff] [review]
patch v1.0 c++
What happens if you have the usemap attribute without the #?
usemap="hello" should work like usemap="#hello", shouldn't it? Does this patch
preserve that behavior?
Reporter | ||
Comment 5•23 years ago
|
||
Fabian: from my perspective usemap="hello" is NOT an URI targeting
<map name="hello"> ; usemap="#hello" is. Is there a quirk I am not aware of that
makes usemap="hello" working necessary ?
Comment 6•23 years ago
|
||
Comment on attachment 70719 [details] [diff] [review]
patch v1.0 c++
just tested IE, usemap="hello" doesn't work either.
so r=fabian
Attachment #70719 -
Flags: review+
filing bugs against browser-general without doing a quick search for open bugs
containing the logical relevant word is mean. please carry your work to the
nice established bug.
*** This bug has been marked as a duplicate of 1882 ***
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → DUPLICATE
Comment 8•23 years ago
|
||
I am not quite sure if this is a dup, because 1882 does, as I understand it,
want support for imagemaps in other files, while this bug is for making maps
work in the same document with the filename fragment.
(also sending to layout component where 1882 is - though I can't seem to change
the owner of resolved bugs)
Component: Browser-General → Layout
Comment 9•23 years ago
|
||
ok. other bugs about this exact issue have been duped to 1882 (e.g. Bug 102851).
verifying.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•