custom key bindings no longer possible
Categories
(Core :: DOM: UI Events & Focus Handling, defect, P5)
Tracking
()
People
(Reporter: khamilton, Unassigned)
References
(Blocks 1 open bug)
Details
(Keywords: helpwanted)
![]() |
||
Comment 1•22 years ago
|
||
Comment 2•22 years ago
|
||
Comment 3•22 years ago
|
||
Comment 4•22 years ago
|
||
Updated•22 years ago
|
Comment 5•22 years ago
|
||
Comment 6•22 years ago
|
||
Comment 7•22 years ago
|
||
Comment 8•22 years ago
|
||
Comment 10•21 years ago
|
||
Updated•21 years ago
|
Comment 11•21 years ago
|
||
Comment 12•19 years ago
|
||
Comment 13•19 years ago
|
||
Comment 14•19 years ago
|
||
Comment 15•19 years ago
|
||
![]() |
||
Comment 16•19 years ago
|
||
Comment 17•19 years ago
|
||
Comment 18•19 years ago
|
||
Reporter | ||
Comment 19•17 years ago
|
||
Comment 20•17 years ago
|
||
Comment 21•16 years ago
|
||
Comment 22•16 years ago
|
||
Comment 23•16 years ago
|
||
Comment 24•16 years ago
|
||
Updated•16 years ago
|
Comment 25•15 years ago
|
||
Comment 26•14 years ago
|
||
Comment 27•14 years ago
|
||
Comment 28•12 years ago
|
||
Comment hidden (advocacy) |
Assignee | ||
Updated•6 years ago
|
Comment 30•3 years ago
|
||
Considering this old archived documentation, I will assume that there may have been changes on this part of the browser (custom key bindings).
@hsinyi, do you think you could redirect me to some more up-to-date documentation or someone who has extensive information on this area?
I would like to re-confirm this bug, part of the Open Major Bug triage, or close it if key binding customization is working as intended.
Can you help me?
Comment 31•3 years ago
|
||
This is definitely still a problem, and has gotten worse in the last few years. For quite a while, it was at least possible to set or disable some of the hardwired key bindings by unpacking omni.ja, modifying browser-sets.inc and re-packaging omni.ja. But sometime around late 2018/early 2019, those files got reorganized and I have never found a way since then to customize or remove any key bindings, especially that Ctrl-W which is forever losing work by closing tabs unexpectedly as described by David A Thompson in comment #23. Sometimes Ctrl-W deletes the last word, sometimes it closes the tab, depending on what component has focus, and I've found no way in current firefox to prevent that tab-closing behavior.
Comment 32•3 years ago
•
|
||
But sometime around late 2018/early 2019,
I think that it's related to bug 1550058 or something which moved shortcut key definitions from XUL. Currently, there is no strict way to customize shortcut keys which are defined by C++ code because:
- The most preferred command for a key combination is, firstly registered in the shortcut key chain.
- There is no way to remove/update registered shortcut keys.
However, there is a hacky way to override existing shortcut keys, that is, build-in shortcut keys are handled at keypress
(which is always fired internally even if non-printable key combination), but XUL <key>
element can be defined with event="keydown"
. Then, it's handled at keydown
event which is a preceding event of keypress
event. However, this may override some other key event handlers such as add-ons, autocomplete, etc and some web apps which Gecko dispatches non-printable keypress
events for the backward compatibility (these prefs may be updated without updating Firefox version).
Comment 33•3 years ago
|
||
Ah, so, the definition files were removed by the changes. Therefore, customizing any files does not help due to no users...
Comment 34•3 years ago
|
||
Reconfirming this bug based on the last 3 comments.
If further testing is required on this part, detailed documentation of what needs to be done should be provided.
FYI: We don't have basic information on how this should work in the first place.
Comment 35•3 years ago
|
||
Currently, shortcut keys are stored internally using a forward linked list of KeyEventHandler
class which is managed by GlobalKeyListener
. Although the scan of looking for updating shortcut key is slow, but this simple structure makes the startup performance better. So perhaps, adding new API of updating shortcut key via command or <key>
element is the first step.
Updated•3 years ago
|
Comment 37•3 years ago
|
||
The severity field for this bug is relatively low, S3. However, the bug has 61 votes and 64 CCs.
:hsinyi, could you consider increasing the bug severity?
For more information, please visit auto_nag documentation.
Updated•3 years ago
|
Comment 38•3 years ago
|
||
I concur the severity of the bug should be increased. Several times a week I lose work because of the Ctrl-W
reason mentioned in comment 23.
Description
•