Closed
Bug 38591
Opened 25 years ago
Closed 25 years ago
Keyboard Shortcuts with Shift Keys don't work on linux.
Categories
(Core :: XUL, defect, P4)
Tracking
()
People
(Reporter: skasinathan, Assigned: saari)
Details
(Keywords: platform-parity, relnote)
Attachments
(1 file)
668 bytes,
patch
|
Details | Diff | Splinter Review |
The keyboard shortcuts that has mutiple keys (Alt+Shift+R for Reply All) doesn't
work on linux. It works fine on Windows and Mac. The string is displayed in
menuitem as "Shift+Alt+R" which I think should be displayed as "Alt+Shift+R".
Here is the XUL code
<key id="key_replyall" key="&replyToAllMsgCmd.key;" xulkey="true" shift="true"
observes="cmd_replyall"/>
Platform:
Linux only.
Comment 1•25 years ago
|
||
Also:
Alt+Shift+N opens a new navigator instead of composer.
Comment 2•25 years ago
|
||
Comment 3•25 years ago
|
||
reassigning to saari as p4 for m18
Assignee: trudelle → saari
Priority: P3 → P4
Target Milestone: --- → M18
Comment 4•25 years ago
|
||
By making this a P4 M18 you're telling me that it's unlikely this is going to
get fixed. If that is going to be the case then we need to change our specs.
Comment 5•25 years ago
|
||
Not at all, I'm only saying that it is a low-priority defect that does not have
to be fixed until after PR2. Do you disagree?
Comment 6•25 years ago
|
||
No, I don't disagree it's post B2. My understanding of the milestones was that
M18 was the last milestone and since this is a P4 it seemed like this would be
one of the last bugs worked on therefore making it one of the first to get cut.
I was just saying that if that's going to happen, assuming there's room on the
keyboard, we could come up with another accelerator combo.
Comment 7•25 years ago
|
||
Only Shift seems to be the problem. Control+Alt together are find. I think see
the problem and will post a patch as soon as I can build&test it.
Comment 8•25 years ago
|
||
Assignee | ||
Comment 9•25 years ago
|
||
excellent, thanks! I'll test it and check it in. I believe MacOS and Windows
still have the same problem, just incase you're feeling really ambitious :-)
Target Milestone: M18 → M16
Comment 10•25 years ago
|
||
Last time I tried it worked fine on windows (NT4). I will test again.
I can't test on a Mac, but if the bug exists there I can try looking at the
code.
Comment 11•25 years ago
|
||
saari, can you look at the comment I added. Where I can find the spec for
Control+A vs Control+Shift+A (XBL docs claim case insensitivity, so I am not
sure if that matters). It seems the Windows/Mac don't have such code.
Comment 12•25 years ago
|
||
Moving all bugs that are not dogfood+, nsbeta2+,features, or nsbeta2- to M21
Target Milestone: M16 → M21
Comment 13•25 years ago
|
||
cc myself and Akkana; update summary; add keywords
Chris--Pike (Axel Hecht) tested this on his linux build and it works for him.
Comment 14•25 years ago
|
||
Hi,
tested the attached patch on solaris, using the meta instead of alt modifier,
I can now use the keybindings.
The order of the modifiers as displayed in the menus is still wrong, though.
Axel
Assignee | ||
Comment 15•25 years ago
|
||
This is a dup, nsbeta3 until I can resolve all the modifier key bugs into one bug
Keywords: nsbeta3
Comment 16•25 years ago
|
||
resolve this bug as a duplicate of 46004 since that bug has already been fixed
*** This bug has been marked as a duplicate of 46004 ***
Status: NEW → RESOLVED
Closed: 25 years ago
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•