35741 - shift key event is not recognized

Status: RESOLVED FIXED

(Core :: XUL, defect, P3)

Description

[Pete Collins (MDG)]

Dave, i am not sure if this is part of the key stuff you currently have
disabled?

test case:

<key id="key_home" xulkey="true" key="H" shift="true" observes="cmd_home" />
<broadcaster id="cmd_home" oncommand="dump('this should work')" />

shift+alt+H

does not respond

where if i set shift="false", no probs.

Please mark invalid if you have this currently disabled.

thanks

pete

Comment 1

[Blake Ross]

Attachment: testcase

Comment 2

[Blake Ross]

adding to cc list (bugzilla should let you make other bug changes and submit an 
attachment at the same time)

Comment 3

[David Hyatt]

Not XBL... XUL key issue... reassigning to saari

Comment 4

[saari (gone)]

shift modifiers are still broken across platforms. Not getting to it in beta2

Comment 5

[Peter Trudelle]

mass-moving all bugs to m21 that are not dofood+, or nsbeta2+

Comment 6

[saari (gone)]

Need to fix modifier keys before we ship

Comment 7

[Peter Trudelle]

nsbeta3-, not known to affect any NS6 features.

Comment 8

[saari (gone)]

This got fixed with the XUL->XBL keybinding conversion

Comment 9

[Pete Collins (MDG)]

Yep, i can verify this works fine . . .

Here is the updated test case:

<commands>
 <commandset id="cmd_home" oncommand="dump('this should work');" />
</commands>

<keyset>
<key id="key_home" xulkey="true" key="H" observes="cmd_home" modifiers="accel,
shift" />
</keyset>

<text value="fuBar" key="key_home" observes="cmd_home" />

--pete