9140 - {html4} AREA elements don't fire events

Status: VERIFIED FIXED

(Core :: DOM: UI Events & Focus Handling, defect, P3)

Description

[luch]

In <map> html command, there used to be the onMouseOver, onMouseOut and others in
the definition of <area> tag.

On the url submited, there's a menu at the left site that uses roll-overs.  When on normal buttons,
the top menu image switches to a new one.  But when on the "I" button, the browser is expected
to make the top image change.  Here is the rollover command used on that page:

<MAP NAME="haut"><AREA SHAPE=RECT COORDS="8,4,17,13" HREF="main.htm" onMouseOver="anim('1');" onmouseout="unanim();"></MAP>

and the image tag which uses that map:

<IMG SRC="pitonbas.gif" usemap="#bas" border=0 width=130>

I'm using the Milestone 7 Mozilla browser.

Hope I didn't forget anything!!
Great work guys!

Comment 1

[cbegle]

Event Handling bug

Comment 2

[Antti Näyhä]

Attaching a minimized test case & rewriting the bug report in accordance
to the guidelines.


Overview description:
  AREA element doesn't fire onMouseOver and onMouseOut events.

Steps to reproduce:
  1) View the attachment.
  2) Move the pointer over and out areas 1-6.

Actual results:
  Nothing.

Expected results:
  Moving in or out from each AREA should produce a JavaScript alert, eg.
  "Pointer over Area 4" or "Pointer out from Area 2".

  (Note that tests 4-6 use nonstandard area shapes, but we'll probably want
  to implement those as well since they are supported by popular browsers.)

Build date & platform:
  - originally reported on Windows 95, build date unknown.

Additional builds & platforms:
  - Apprunner 1999072711 (Windows NT 4.0 SP4)
  - Viewer 1999072711 (Windows NT 4.0 SP4)
  - M8 Apprunner (Windows NT 4.0 SP4)

Comment 3

[Antti Näyhä]

Attachment: Minimized test case

Comment 4

[Antti Näyhä]

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

Comment 5

[Antti Näyhä]

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

Comment 6

[Prashant Desale]

Tested with todays builds. Its still there.

Comment 7

[Prashant Desale]

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

Comment 8

[joki (gone)]

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

Comment 9

[Adam D. Moss]

Still there.  I've changed OS to 'All'.

I understand that this is quite an important omission,
since HTML 4.0 compliance requires support of this feature
(I am told).

Comment 10

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

HTML 4.0 compliance doesn't require much of anything...

Comment 11

[Hixie (not reading bugmail)]

dbaron: how true, how true... :-(

A better way of phrasing it is "This is required for full HTML4 support".
Adding relevant dependencies.

Comment 12

[ekrock's old account (dead)]

Moving M16.

Comment 13

[ekrock's old account (dead)]

Bulk moving [testcase] code to new testcase keyword. Sorry for the spam!

Comment 14

[joki (gone)]

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

Comment 15

[joki (gone)]

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

Comment 16

[joki (gone)]

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

Comment 17

[brian]

Attachment: Full testcase testing all 12 events

Comment 18

[brian]

There are 12 events that area elements should support according to the W3C 
Recommendation:
http://www.w3.org/TR/html4/struct/objects.html#adef-usemap

They are demonstrated in the recently added attachment

Comment 19

[joki (gone)]

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

Comment 20

[vidur (gone)]

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

Comment 21

[vidur (gone)]

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

Comment 22

[vidur (gone)]

Nominating for beta 1. Look at the number of DUPs on this one.

Comment 23

[Alan S. Jones]

If this has really been moved to beta1 then shouldn't the milestone be set to 
m14 or m15?

Comment 24

[daver]

adding [PDT+]

Comment 25

[vidur (gone)]

Changing milestone.

Comment 26

[Mike McCabe]

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

Comment 27

[joki (gone)]

Okay, vidur say's he thinks this should fully work now.  Resolving fixed

Comment 28

[brian]

ondblclick event still doesn't fire. See attachment 4647 [details]. Also, you cannot tab 
to area elements (so keypress events cannot be tested), but that, I assume, is a 
separate bug.

Comment 29

[brian]

I have verified that the ondblclick event does not fire for IMG or A elements so 
this is a separate bug. This bug is therefore resolved.

Comment 30

[Jan Carpenter]

Not fixt. 
using the attachment 
http://bugzilla.mozilla.org/showattachment.cgi?attach_id=4647

Win98:  2000021408
- onmouseout creates repeating alert dialog boxes which after efforts to close 
them crash the browser

Linux: 2000021416
- none of the tests work

Mac: 
- ondblclick doesn't work
- onmouseout creates repeating alert messages which quickly lock up the program, 
and eventually crash the system.
- onmouseover = same

Comment 31

[joki (gone)]

Okay, fixed the repeating alert message problem.  The dblclick problem is 
covered elsewhere.  This is as far as this gets for M14.  The only remaining 
problem I see below that would keep this bug open is that part about this not 
working at all on Linux.  This is all xp code though, it really should work.  
Removing PDT+, moving to M15.  If someone with a working Linux build can verify 
this is still a problem I'd appreciate it with any additional info available.  
If we can verify this works on Linux this bug should be closed.

Comment 32

[leger]

Putting on the PDT+ radar for beta1.  But please mark this one fixed and open 
additional bugs if necessary.

Comment 33

[joki (gone)]

Fine.  Since as far as I know this should work fine on linux too i'm just going 
to mark it fixed then. 

Comment 34

[Prashant Desale]

Reopening bug. I checked it with todays builds [02-24-09].

Not fixed. 
using the attachment 
http://bugzilla.mozilla.org/showattachment.cgi?attach_id=1032

Win95:  2000022409
- onmouseout creates repeating alert dialog boxes which after efforts to close 
them crash the browser

Linux: 2000022409
- mouseout event does not fire.

Comment 35

[joki (gone)]

The linux part of this is known and not a big deal.  The Win95 bit is new and 
the only part of this that might make this PDT+.  So is the suggestion here that 
it only fail on Win95?  Since the fix was developed on WinNT I'm asusming it 
still works there.  My build is a day or two old so I'm repulling to check.  But 
can someone verify that?

Comment 36

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

What's the bug# on the Linux problem?

Comment 37

[joki (gone)]

I hadn't realized that was a different test case.  With the original one it 
works fine but with the one mentioned it does crash on WinNT as well.

I hadn't opened a seperate bug for the linux stuff since the last mention before 
this said the test case didn't work at all on linux which I was skeptical of and 
waiting for additional verfication.  The most recent comment says mouseout 
doesn't work which at least indicates the other stuff works.  That still seems a 
little odd to me, though.  it would be interesting to rewrite this test with 
'dump' instead of 'alert' and see what happens.

Comment 38

[Jim Roskind]

Can we have an update, and a planned landing date?
I'm adding the comment to the status whiteboard that we'll have to PDT- this on
3/3.  Time is running out...

Comment 39

[joki (gone)]

Okay, I have worked out a fix that doesn't scare me at all like a couple of 
others did.

Comment 40

[Kevin McCluskey (gone)]

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

Comment 41

[joki (gone)]

Okay.  I've checked in a fix for the most recent problem, that of window level 
mouseover events which were actually occurring over the alert box constantly 
triggering the mouseexit handler direclty beneath the alert and causing a loop. 
I'll need to fix this a bit more later by differentiating these two types of 
events so we don't get them in content but that's a larger fix and can wait for 
post-beta.

Removing pdt+, removing beta1.  

Comment 42

[Doron Rosenberg (IBM)]

this was working before, but on the latest build, seems to be gone (at least for 
netscape.com)

Comment 43

[leger]

Putting on the PDT- radar for beta1.

Comment 44

[Jan Carpenter]

adding beta2 keyword, and removing pdt status

Comment 45

[Paul MacQuiddy]

removing beta1 keyword so it doesn't show up on beta1 radar

Comment 46

[joki (gone)]

Mass-moving bugs out of M15 that I won't get to.  Will refit individual 
milestones after moving them.

Comment 47

[leger]

PDT would like a retest with latest build please.  Thanks!

Comment 48

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

Filed crash bug on second testcase as bug 37479.  Some of the events in that
testcase still don't work.  However:
 * some of them shouldn't work (key/focus events)
 * For the ones that don't, do they work on other elements?  (I don't know.) 
If  they don't, are appropriate bugs filed?

Comment 49

[Jan Carpenter]

using attachment 4647 [details]:

ondblclick doesn't work on any platform
onmousemove locks up the browser on the mac 

2000-05-05-08-M16  Win32
2000-05-05-13-M16  Linux
2000-05-05-10-M16  Mac

Comment 50

[leger]

Removing [RETEST NEEDED] for PDT re-eval ...thanks!

Comment 51

[Nisheeth Ranjan]

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

Comment 52

[leger]

Putting on [nsbeta2+] radar for beta2 fix. 

Comment 53

[harishd]

Taking from Joki for analysis.

Comment 54

[harishd]

All events except "onDblClick" [ 06/13/00 build ]. And "onMouseMove" did not 
lock up mac [ 06/12/00 build ]. Jan, could you verify this? Thanx.

Comment 55

[harishd]

All events except "onDblClick" [ 06/13/00 build ] work. And "onMouseMove" did 
not lock up mac [ 06/12/00 build ]. Jan, could you verify this? Thanx.

Comment 56

[ckritzer (gone)]

Checked this out on MacOS9 2000-061908-M17 build and all events except dblclick
and onselect work.

Harish, check out http://voodoolady/events/mousies for testcases related to
mouse and key events.  I'll try to post those to an external website as well for
folks outside the firewall...

Comment 57

[harishd]

From ckritzer's comment I understand that we don't freeze on MAC anymore. 

Events not working:
 OnSelect ( Ref. Bug 4033 )
 OnDblClick (Ref. Bug 29473 )

Tom, what should we do with this bug?

Comment 58

[harishd]

Okay, the bug seems to be fixed.

Btw, OnSelect is not supposed to work on IMG maps...and onDblClick will be dealt 
as a different bug ( probably 29473 ).  

Closing the bug.

Comment 59

[Jan Carpenter]

I'm going to verify this bug with the note that there are still a couple of 
things happening wrong. I think they are new bugs, so I'm going to write them up 
in a new bug and ref this bug in it.

2000-07-12-11-M17 : Linux
onMouseOver brings up 2 alert boxes

2000-07-12-13-M17 : Mac
onMouseout & onMouseOver bring up 2 alert boxes

2000-07-12-09-M17 : WinNT & Win98
Verified 

(onDoubleClick is indeed covered on bug 29473)