25199 - Opening an imagemap link in a new window/tab doesn't send coordinates

Status: NEW

(Core :: Layout: Images, Video, and HTML Frames, defect, P3)

Description

[Steven de Hoog]

When clicking on an imagemap, the coordinates get sent as part of the URL.
However, selecting 'open in new window' from the context menu does not.

For example: left-clicking goes to  imgmaptest.html?xpos,ypos
             right-clicking goes to imgmaptest.html

Expected behaviour: include the coordinates when opening in a new window the
same way left-clicking does.

Comment 1

[Steven de Hoog]

Attachment: Simple HTML file with imagemap

Comment 2

[Steven de Hoog]

Attachment: and an image to test it with...

Comment 3

[pnunn]

Eric:
Do you know who handles image maps in layout land?
-p

Comment 4

[Eric Pollmann]

Well, imagemaps are definitely handled in layout land (nsImageMapFrame.cpp or
somesuch).  I think Kipp was the one who implemented these, but last I heard
nobody owned them.  I'll take this bug for now but it may not get fixed for a
while (my buglist, ack, is huge).

Comment 5

[Eli Goldberg]

QA Assigning to Mr. Peterson, who is now handling image maps.

Comment 6

[Eric Pollmann]

Rescheduling

Comment 7

[ekrock's old account (dead)]

Nom. nsbeta2, recc. nsbeta2+. This is basic HTML 3.2, and image maps are widely 
used for navigation, and we aren't passing the URL between windows correctly. 
Very bad Basic Browser Correctness issue. HTML 3.2 is key b2 criterion.

Comment 8

[leger]

Putting on [nsbeta2-] radar. Not critical to beta2.  But adding to nsbeta3 
keyword radar to get a fix in before rtm.

Comment 9

[Eric Pollmann]

This bug has been marked "future" because the original netscape engineer working 
on this is over-burdened. If you feel this is an error, that you or another 
known resource will be working on this bug,or if it blocks your work in some way 
-- please attach your concern to the bug for reconsideration.

Comment 10

[Kevin McCluskey (gone)]

Marking nsbeta3-

Comment 11

[Gervase Markham [:gerv]]

It seems unclear to me whether this bug requires either of a "developer" or 
"user" release note. If anyone feels it does, can they please draft one and then 
nominate with the relnote-user or relnote-rtm strings in the Status Whiteboard.

Thanks :-)

Gerv

Comment 12

[Hixie (not reading bugmail)]

Upon managerial request, adding the "testcase" keyword to 84 open layout bugs that
do not have the "testcase" keyword and yet have an attachement with the word
"test" in the description field. Apologies for any mistakes.

Comment 13

[Tim Powell]

Attachment: testcase of Washington Tech image map

Comment 14

[Tim Powell]

I added a testcase taken from HTML on the Washington Tech website. I first saw
it on this article: http://www.washtech.com/news/media/11421-1.html It appears
that the right-click only looks at the A href instead of the ismap information
in the Img tag.

Adding Mozilla1.0 and 4xp keywords (Netscape 4.x does not exhibit this problem).

Comment 15

[Tim Powell]

Bug 82826 says opening in a new window by middle-clicking also fails. Should it
be a duplicate? 

Ctrl+click on build 2001072503 behaves strangely. The new window opens and shows
the url without the coordinates, but the click apparently also goes to the
original window and shows the proper results (using the coordinates) in the
original window.

Comment 16

[Kevin McCluskey (gone)]

Bulk reassigning form bugs to Alex

Comment 17

[Jeremy M. Dolan]

*** Bug 82826 has been marked as a duplicate of this bug. ***

Comment 18

[Jeremy M. Dolan]

82826 was assigned to joki and 097 targeted. Happening for tabs too.

Comment 19

[David D. Kilzer (ddk)]

More fun with imagemaps and opening new windows:

1. Click on "Show dependency graph" link on this bug in Bugzilla.
2. Right-click, select "Open in new window" (or middle-click) on the
   imagemap.
3. Note that source for imagemap is returned from webdot.cgi on
   www.research.att.com.

Fun stuff.

Comment 20

[Mike Pinkerton (not reading bugmail)]

*** Bug 152489 has been marked as a duplicate of this bug. ***

Comment 21

[Boris Zbarsky [:bzbarsky]]

The problem is merely that of getting the click coords in the coordinate system
of the image.  Unfortunately, that's a pretty hard problem (since there is no
real way, from where the new window/tab code lives, to determine where the image
is located in the viewport or document).

I suppose we could add an interface to events that would not have classinfo and
would report the coords in the target frame (whatever that means in general).

Comment 22

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Isn't this a question of how we dispatch links?  Shouldn't we resolve
server-side image maps in a layer before we separate open-in-new-window from
normal open?

Comment 23

[Boris Zbarsky [:bzbarsky]]

"open in new window/tab" are hooked up in the UI itself (in nsContextMenu.js). 
They basically share no code with the normal click codepath....

Comment 24

[Daniel Clemente]

*** Bug 245863 has been marked as a duplicate of this bug. ***

Comment 25

[campbell]

This has been a cronic problem since the gg with Mozilla.  I'm not sure
this is the same bug, but successive clicks on an image map will result
only in the FIRST set of coords being resent.  This problem does not
Occur in IE, Opera, or Netscape prior to V5.

Dhu

Comment 26

[Ryan VanderMeulen [:RyanVM]]

Interestingly enough, both IE7 and Opera 9.02 appear to fail on these testcases as well.

Comment 27

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Would probably be fixed by the plan in bug 335963.

Comment 28

[Daniel Bodea [:danibodea]]

This issue still occurs just the same on Windows 10, Ubuntu 20, Mac OS 11.

Comment 29

[Daniel Holbert [:dholbert]]

Notes on various ways to open-in-new-tab:
A) Ctrl+Click adds the coordinates the current tab's URL (rather than the one in the new tab).
B) Middle-click doesn't put the coordinates in either URL.
C) Right-click + open-in-new-tab doesn't put the coordinates in either URL.

Chrome puts the coordinates in the new tab's URL (as-expected, I think?) for (A) and (B), but they match us for (C) with the context-menu (no coordinates).

Anyway: seems like an interesting behavioral difference still, but pretty low-severity.

Comment 30

[BugBot [:suhaib / :marco/ :calixte]]

The severity field for this bug is relatively low, S4. However, the bug has 3 duplicates.
:emilio, could you consider increasing the bug severity?

For more information, please visit auto_nag documentation.

Comment 31

[Emilio Cobos Álvarez (:emilio)]

Imagemaps aren't so wildly used nowadays, S4 seems reasonable.