39721 - Switching keyboard layout with alt containing keystroke activates menu

Status: RESOLVED WORKSFORME

(Core :: XUL, defect, P3)

Description

[Mikhail Khlopotov]

From Bugzilla Helper:
User-Agent: Mozilla/4.7 [en] (Win98; I)
BuildID:    20000515

If keyboard layout switching sequence contains Alt (suct as alt-shift)
switching layout activates menu.

Reproducible: Always
Steps to Reproduce:
1. Press Alt-Shift
2. See that menu activated
3.

Actual Results:  After pressing Alt-Shift menu got active

Expected Results:  Keyboard combinatiob of Alt and other special keys should be 
ignored, not interpreted as alt.

This behaviour does not pervents from switching layout, but is 
annoying when typing mixed langage texts, etc.

Comment 1

[Mike Pinkerton (not reading bugmail)]

ohhhh hyatt babieeeeeee

Comment 2

[sairuh (rarely reading bugmail)]

*spam*: transferring current XP Menu bugs over to jrgm, the new component owner.
feel free to add me to the cc list (unless am the Reporter) of any of these, if
you have any questions/etc.

Comment 3

[Asa Dotzler [:asa]]

setting to NEW

Comment 4

[Akkana Peck]

I'm not sure what's causing this, but if anyone's really having problems with
this, you might want to try adding this to your prefs.js:

user_pref("ui.key.menuAccessKeyFocuses", false);

and see if it helps as a workaround.

Comment 5

[Simon Montagu :smontagu]

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

Comment 6

[Simon Montagu :smontagu]

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

Comment 7

[Horsng Junn]

It happens all around the mozilla, not just in the browser.
In #34237 it's sent to BIDI, but I don't think this bug belogs there either.

I'm using Korean WIN2000, so maybe it's I18N issue.
But I think English only users can reproduce this behavior with following steps.
(I didn't tested this in English WIN2000. So it may not work.)

Steps to Reproduce:
1. Open "Keyboard" control panel.
2. In Locale (or whatever, I'm using Korean WIN2000 ;-) tab,
     press the "Append..." Button.
3. Append an input locale. Maybe Dvorak?
4. Make sure "locale transition" shortcut is assigned to left alt+shift (the
default)
5. In anywhere in mozilla, press left alt+shift.

The keyboard layout changes as expected, but mozilla focuses File menu.

As I said above, this may not work (as buggily like reported) in English WIN2000.
Anyway in my WIN2000 it happens when I change between default MS IME (Qwerty /
Korean) and Dvorak English,
not when I change between Korean / English.
So it seems to me like a simple "alt" key capture problem, not an I18N issue.

And the suggested
user_pref("ui.key.menuAccessKeyFocuses", false);
workaround does work for me.

PS. Is this comment readable?
I've been hesitating to report bugs because of my poor English.

Comment 8

[kmike]

really annoying bug...
i don't have win32 box in vicinity right now to test Akkana's workaround.

Comment 9

[Simon Montagu :smontagu]

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

Comment 10

[Simon Montagu :smontagu]

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

Comment 11

[Andreas Becker]

Nominating as nsbeta1 since this appears to be an annoying bug and many 
duplicates are filed. This bug got futured in 2000-06-19, respectfully 
requesting a target milestone change.

Comment 12

[timeless]

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

Comment 13

[Simon Montagu :smontagu]

Attachment: Patch to fix the bug

Comment 14

[Simon Montagu :smontagu]

The attached patch fixes this bug on Windows. Can someone confirm that it
doesn't break other platforms?

Comment 15

[Peter Trudelle]

Adding patch keyword, clearing target for triage.

Comment 16

[David Hyatt]

Menus. --> pinkerton.

Comment 17

[Jesse Ruderman]

See also bug 79898, Alt+accesskey activates menu bar if alt released before 
accesskey.

Comment 18

[Jaime Rodriguez, Jr.]

Marking as CatFood because it is "annoying when typing mixed langage texts, 
etc.", and very inconvenient (i.e. not efficient).

Request a M0.9.2 TM.

Is the patch as good one?

Comment 19

[Peter Trudelle]

Doesn't fit the profile for 0.9.2, but could still possibly take the patch
during the 'limbo' period.

Comment 20

[Jesse Ruderman]

I forgot about the patch here, but anyway there's now a (different) patch on bug
79898, also waiting for review.

Comment 21

[Simon Montagu :smontagu]

Attachment: New patch incorporating the patch in attachment 38017 to bug 79898

Comment 22

[Mike Pinkerton (not reading bugmail)]

a new patch with http://bugzilla.mozilla.org/showattachment.cgi?attach_id=38116
needs to be written. 

Also, I don't like |if ( const == var )|. It's too hard to read. Use |if ( var
== const )|.

Comment 23

[Mike Kaply [:mkaply]]

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

Comment 24

[Jesse Ruderman]

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

Comment 25

[Simon Montagu :smontagu]

It looks as if this bug has been fixed along with bug 79898. Is anyone still
seeing this one?

Comment 26

[Claude Gohier]

Seems to be fixed in 2001072503 trunk.

Comment 27

[Spiros Ioannou]

Works fine now.

Comment 28

[Mike Pinkerton (not reading bugmail)]

--> jag

Comment 29

[jag (Peter Annema)]

Mikhail Khlopotov, it looks like this has been fixed, I'm marking this bug
"Works For Me" based on the comments in this bug and not being able to reproduce
it myself. If you are still seeing this problem, please reopen this bug.