Opened 8 years ago

Closed 8 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)

4262.patch (1.6 KB) - added by Tobiasz Cudnik 8 years ago.
CM_Shift+F10.jpg (246.8 KB) - added by Pranav 8 years ago.
Cm_Ctrl+Shift+F10.JPG (186.3 KB) - added by Pranav 8 years ago.

Download all attachments as: .zip

Change History (14)

comment:1 Changed 8 years ago by Tobiasz Cudnik

Milestone: CKEditor 3.0CKEditor 3.1

comment:2 Changed 8 years ago by Tobiasz Cudnik

Owner: set to Tobiasz Cudnik
Status: newassigned

comment:3 Changed 8 years ago by Tobiasz Cudnik

Resolution: invalid
Status: assignedclosed

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.

comment:4 in reply to:  3 ; Changed 8 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 8 years ago by Tobiasz Cudnik

Attachment: 4262.patch added

comment:5 in reply to:  4 ; Changed 8 years ago by Tobiasz Cudnik

Resolution: invalid
Status: closedreopened

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 8 years ago by Tobiasz Cudnik

Keywords: Review? added

Changed 8 years ago by Pranav

Attachment: CM_Shift+F10.jpg added

Changed 8 years ago by Pranav

Attachment: Cm_Ctrl+Shift+F10.JPG added

comment:7 in reply to:  5 Changed 8 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 8 years ago by Tobiasz Cudnik

Wrong position is already covered by #3528 and fix is available in SVN.

comment:9 Changed 8 years ago by Garry Yao

Resolution: wontfix
Status: reopenedclosed

@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 8 years ago by Garry Yao

Keywords: Review+ added; Review? removed
Resolution: wontfix
Status: closedreopened

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 8 years ago by Tobiasz Cudnik

Resolution: fixed
Status: reopenedclosed

Fixed with [4177].

Note: See TracTickets for help on using tickets.
© 2003 – 2017 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy