Opened on 10/17/2016 at 03:45:04 PM
Closed on 10/31/2016 at 02:09:00 PM
#4544 closed change (fixed)
Consider platformHTMLBindings.xml when detecting available keyboard shortcuts
Reported by: | trev | Assignee: | trev |
---|---|---|---|
Priority: | P3 | Milestone: | |
Module: | Automation | Keywords: | |
Cc: | Blocked By: | ||
Blocking: | Platform: | Unknown / Cross platform | |
Ready: | yes | Confidential: | no |
Tester: | Unknown | Verified working: | no |
Review URL(s): |
Description
Background
keySelector.js currently looks for <xul:key> elements in the browser window when determining which shortcuts are already taken. This check misses Accel-Shift-V key however ("paste as plain text").
What to change
Take <xbl:handler> elements from chrome://global/content/platformHTMLBindings.xml into consideration as well, this is where Accel-Shift-V key is defined.
Attachments (0)
Change History (5)
comment:1 Changed on 10/17/2016 at 03:45:21 PM by trev
- Summary changed from Consider platformHTMLBindings.xml when detecting available keyboard shortcust to Consider platformHTMLBindings.xml when detecting available keyboard shortcuts
comment:2 Changed on 10/31/2016 at 10:03:04 AM by trev
- Owner set to trev
comment:3 Changed on 10/31/2016 at 10:08:54 AM by trev
- Review URL(s) modified (diff)
- Status changed from new to reviewing
comment:4 Changed on 10/31/2016 at 02:08:23 PM by abpbot
comment:5 Changed on 10/31/2016 at 02:09:00 PM by trev
- Resolution set to fixed
- Status changed from reviewing to closed
Note: See
TracTickets for help on using
tickets.
A commit referencing this issue has landed:
Issue 4544 - Consider platformHTMLBindings.xml when detecting available keyboard shortcuts