Opened 7 years ago
Closed 7 years ago
#4262 closed Bug (fixed)
Tab and Shift+Tab not working to cycle through context menu in IE
| Reported by: | Pranav | Owned by: | tobiasz.cudnik |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.1 |
| Component: | General | Version: | |
| Keywords: | Oracle Review+ | Cc: | Pranav Senthil |
Description
Open context menu by pressing Ctrl+Shift+F10. Press Tab or Shift+Tab to navigate through context menu. You will observe that these hotkeys are not working. Issue is present in IE only.
Attachments (3)
Change History (14)
comment:1 Changed 7 years ago by tobiasz.cudnik
- Milestone changed from CKEditor 3.0 to CKEditor 3.1
comment:2 Changed 7 years ago by tobiasz.cudnik
- Owner set to tobiasz.cudnik
- Status changed from new to assigned
comment:3 follow-up: ↓ 4 Changed 7 years ago by tobiasz.cudnik
- Resolution set to invalid
- Status changed from assigned to closed
comment:4 in reply to: ↑ 3 ; follow-up: ↓ 5 Changed 7 years ago by Pranav
Replying to tobiasz.cudnik:
Correct shortcut for context menu is SHIFT+F10.
Using CTRL+SHIFT+F10 causes the reported result, although rebinding context menu to this key combination in CKEDITOR.config.keystrokes make it work again.
Shift+F10 opens the context menu outside the editor area. To open the context menu inside the editor area, we have to use Ctrl+Shift+F10. But in this case we are not able to navigate. I think, this issue should be fixed.
Changed 7 years ago by tobiasz.cudnik
comment:5 in reply to: ↑ 4 ; follow-up: ↓ 7 Changed 7 years ago by tobiasz.cudnik
- Resolution invalid deleted
- Status changed from closed to reopened
Replying to Pranav:
Shift+F10 opens the context menu outside the editor area. To open the context menu inside the editor area, we have to use Ctrl+Shift+F10.
What do you mean by "opens context menu outside the editor area" ? When focus is inside editor area, pressing SHIFT+F10 results in showing the menu outside editor's area ? Or SHIFT+F10 doesn't work inside editor's area ?
For me SHIFT+F10 works as it should, which i've confirmed on IE6 and IE8.
I'm attaching the patch which simply extends default config to handle Ctrl+Shift+F10 directly.
comment:6 Changed 7 years ago by tobiasz.cudnik
- Keywords Review? added
Changed 7 years ago by Pranav
Changed 7 years ago by Pranav
comment:7 in reply to: ↑ 5 Changed 7 years ago by Pranav
Replying to tobiasz.cudnik:
Replying to Pranav:
Shift+F10 opens the context menu outside the editor area. To open the context menu inside the editor area, we have to use Ctrl+Shift+F10.
What do you mean by "opens context menu outside the editor area" ? When focus is inside editor area, pressing SHIFT+F10 results in showing the menu outside editor's area ? Or SHIFT+F10 doesn't work inside editor's area ?
For me SHIFT+F10 works as it should, which i've confirmed on IE6 and IE8.
I'm attaching the patch which simply extends default config to handle Ctrl+Shift+F10 directly.
I am attaching two screenshots showing behaviour when Shift+F10 and Ctrl+Shift+F10 is pressed.
Shift+F10: http://dev.fckeditor.net/attachment/ticket/4262/CM_Shift%2BF10.jpg
Ctrl+Shift+F10: http://dev.fckeditor.net/attachment/ticket/4262/Cm_Ctrl%2BShift%2BF10.JPG
comment:8 Changed 7 years ago by tobiasz.cudnik
Wrong position is already covered by #3528 and fix is available in SVN.
comment:9 Changed 7 years ago by garry.yao
- Resolution set to wontfix
- Status changed from reopened to closed
@Pranav: In IE, 'contextmenu' event key 'CTRL+SHIFT+F10' is conflicting with the 'F10' hotkey of giving focus to browser's menu bar, it's a browser bug, so if you want to trigger contextual menu at browser default position( instead of at the top-left corner of the document ), use MenuKey instead. @Tobias: I'm afraid we should always preserve OS/Browser native keys as much as possible.
comment:10 Changed 7 years ago by garry.yao
- Keywords Review+ added; Review? removed
- Resolution wontfix deleted
- Status changed from closed to reopened
After talking with Tobias, it's turned out that this keystroke (CTRL+SHIFT+F10) is one of those proprietary keys binding of IE, so we could override to avoid any confusion.
comment:11 Changed 7 years ago by tobiasz.cudnik
- Resolution set to fixed
- Status changed from reopened to closed
Fixed with [4177].
Correct shortcut for context menu is SHIFT+F10.
Using CTRL+SHIFT+F10 causes the reported result, although rebinding context menu to this key combination in CKEDITOR.config.keystrokes make it work again.