37950 - edit fields eat context clicks

Status: VERIFIED FIXED

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

Description

[Dan M]

Perhaps this is intentional, but it's getting in the way of bug 14874. That is, 
you can't attach a context menu to an edit field. Notice that on a web page with 
(text) input fields, the context menu generally shows up when you click in the 
content, but not the input field(s).
  The same problem crops up if you explicitly attach a popup. A "popup" popup 
attached to a text field and, say, a checkbox will show up in both cases. A 
"context" popup works checkbox. And there's the problem. Bug 14874 is the one 
where we want to attach context popups to edit fields.
  Simon said throw this one at Akkana...

Comment 1

[Akkana Peck]

Mike is completely rewriting text control event handling, expected to land in
the next few days.  Marking dependency accordingly.

If you're blocked and desperate for something to try, you might try going to
nsEnderEventListener::MouseDown(), read the comments, and comment out the place
where it returns NS_OK for button 3.  No guarantees on what you'll get, but on
some platforms it might get you something that would work for testing.

Comment 2

[leger]

Putting on [nsbeta2+] radar for beta2 fix.

Comment 3

[leger]

Putting on [6/01] radar.

Comment 4

[Akkana Peck]

Mike has most of Ender Lite (bug 34896) checked in, but disabled.  Several of us
are testing with it turned on, to check for problems.  Can someone please
provide a test case for this bug, so that I can see whether Ender Lite will
solve it?

Comment 5

[rubydoo123]

Akkana, just talked with Kathy, a good test case to use would be the named 
anchor dialog, the text field has a tooltip.

Removing NEED TESTCASE from whiteboard

Comment 6

[Akkana Peck]

Sorry for being dense, but what do right-mouse events have to do with tooltips? 
Should a tooltip come up when I right-mouse over the text field? 

The real question is, are context menus supposedly attached to text fields, so
if I right mouse and the event works, I should see a context menu?  Or are they
disabled waiting for the event problem to be fixed?

Comment 7

[Akkana Peck]

Putting NEED TESTCASE back on whiteboard, waiting to see what tooltips and
context menus have in common.

Comment 8

[rubydoo123]

the initial problem talked about context menus and popup -- aren't tooltips 
popups?

Comment 9

[leger]

Setting to [nsbeta2+][6/15]

Comment 10

[Akkana Peck]

Tooltips are popups, but I thought this bug was that the right-mouse context
click wasn't getting through to text fields (because they were explicitly
disabled due to a problem in nsGfxTextControlFrame).

I don't know how tooltips work, or why they're not working here.  Are you
expecting some kind of enter event that you're not getting?

If this is a problem with popups not working, rather than with events not
getting through, then we should probably call in someone from xpfe who knows
something about how popups work, and change the summary to reflect the real
problem.

Comment 11

[Akkana Peck]

I *think* this will be solved by ender lite.  In today's build, if I right-mouse
over a text field in the browser content area, I get nothing.  In today's build
with ender lite enabled, if I right mouse over a text field in the browser
content area, I get an assert:

###!!! ASSERTION: can't query for shorthand properties: 'Error', file
/builds/enderlite/mozilla/layout/html/style/src/nsCSSDeclaration.cpp, line 3906
###!!! Break: at file
/builds/enderlite/mozilla/layout/html/style/src/nsCSSDeclaration.cpp, line 3906

and then a context menu pops up rooted at the upper left of the content area.

I'm not sure what this CSS error means, but it sounds like it's unrelated to
whether the event gets through.  So it looks like this bug will be fixed when
ender lite is enabled.  (I'm not going to mark it fixed until that happens.)

Comment 12

[Akkana Peck]

Ender lite is in, and when I right-click in a text field, a context menu pops
up.  It pops up in the wrong place and I get a bunch of asserts, but that's
someone else's bug: the events are getting through.  Marking this fixed.

Comment 13

[ckritzer (gone)]

Mass update:  changing qacontact to [e-mail]

Comment 14

[ckritzer (gone)]

Marking VERIFIED FIXED on:
- MacOS9 2000-07-31-04-M17 Commercial
- Linux6 2000-07-29-04-M17 Commercial
- Win98  2000-07-31-04-M17 Commercial