2989 - [EVENTTARG] Blank areas of elements doesn't fire events

Status: VERIFIED FIXED

Description

[Erik Arvidsson]

If I have an element with text in the event isn't fired until the text node
fires the event. I'll show a short sample:

    <div style="background-color: yellow; width: 500px; height: 500px;"
onmouseover="alert('K.')">
        Short text
    </div>

The text node fires the event that is then bubbled to the div node but the div
node doesn't fire an event by itself.

Comment 1

[Paul MacQuiddy]

per leger, assigning QA contacts to all open bugs without QA contacts according
to list at http://bugzilla.mozilla.org/describecomponents.cgi?product=Browser

Comment 2

[gerardok]

QA contact re-assigned according to the product areas we're currently working
on.

Comment 3

[joki (gone)]

The behavior of mouseover isn't really specified to this level.  If I mouseover
the text, I'm going to get a mouseover on the text which will bubble up to the
div.  That's just the way it works at the moment.

There was a separate bug where hitting the rest of the div would not cause an
event at all.  This is fixed now.

Comment 4

[Erik Arvidsson]

This isn't satisfactory from a dhtml developer point of view. This forces me to
fill up the entire container with text which isn't possible (the borders). For
example I can't create buttons, toolbars etc that can be clicked anywhere, only
at the text.

This is almost as bad as when you in NN4.x only provided a background color
behind the text in the container!

Comment 5

[joki (gone)]

No, you still get events on the div.  And if you're not keep in mind I only
marked this fixed today, the change only went in hours ago.  So the deal is

over empty space -> event goes to div
over text -> event goes to text, bubbles to div.

This is the solution.

Comment 6

[Jan Carpenter]

Build: 1999-04-26-08
Not fixt.
I get no alert for this test.  I put the test included in this bug report on
http://slip/projects/marvin/bugs/bug_2989.html for easier reference.

Comment 7

[Erik Arvidsson]

The test doesn't use window.alert. It writes out the the text using DOM and it
worked with 1999-04-22 (maybe something went wrong during those two days?)

I don't see how it becomes more easy when the provided URL doesn't work :-) The
old URL is still available at http://webfx.eae.net/gecko/testEvents.html

Comment 8

[joki (gone)]

Yeah, noticed the change still in my tree.  Must have forgot to check it in.
I'll add it when I can.

Comment 9

[Evan Jones]

Created attachment 1003 [details]
Fires events with a mostly blank DIV tag

Comment 10

[Evan Jones]

It seems to me that the original problem, Blank areas of elements doesn't fire
events, is fixed now. I tested the code in the bug description that was created
by [e-mail] (and I created a test case for the bugathon) on the
daily build for July 26, 1999 on Linux and the blank areas of the DIV fired the
onmouseover events. Can anyone else (maybe a Mozilla engeneer?) confirm if this
is fixed now?

Comment 11

[David Baron :dbaron: 🏴󠁵󠁳󠁣󠁡󠁿 ⌚UTC-8 (if account gets disabled due to email bounces, ask a bugzilla admin to reenable it)]

Marking this resolved-fixed.  I think joki checked in the fix sometime between
April and now.  It sure looks fixed.

This bug dealt only with the second test on the page.  However:

I couldn't find any bugs on the other two tests on the page.  The first one is
IE-specific syntax that has been replaced in DOM2.

The problem in the third test area is a dup of bug 11009 and I will add a note
there.