Ticket #4262 (closed Bug: fixed)

Opened 5 years ago

Last modified 5 years ago

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

4262.patch (1.6 KB) - added by tobiasz.cudnik 5 years ago.
CM_Shift+F10.jpg (246.8 KB) - added by Pranav 5 years ago.
Cm_Ctrl+Shift+F10.JPG (186.3 KB) - added by Pranav 5 years ago.

Change History

comment:1 Changed 5 years ago by tobiasz.cudnik

  • Milestone changed from CKEditor 3.0 to CKEditor 3.1

comment:2 Changed 5 years ago by tobiasz.cudnik

  • Status changed from new to assigned
  • Owner set to tobiasz.cudnik

comment:3 follow-up: ↓ 4 Changed 5 years ago by tobiasz.cudnik

  • Status changed from assigned to closed
  • Resolution set to invalid

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 ; follow-up: ↓ 5 Changed 5 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 5 years ago by tobiasz.cudnik

comment:5 in reply to: ↑ 4 ; follow-up: ↓ 7 Changed 5 years ago by tobiasz.cudnik

  • Status changed from closed to reopened
  • Resolution invalid deleted

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 5 years ago by tobiasz.cudnik

  • Keywords Review? added

Changed 5 years ago by Pranav

Changed 5 years ago by Pranav

comment:7 in reply to: ↑ 5 Changed 5 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 5 years ago by tobiasz.cudnik

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

comment:9 Changed 5 years ago by garry.yao

  • Status changed from reopened to closed
  • Resolution set to wontfix

@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 5 years ago by garry.yao

  • Status changed from closed to reopened
  • Keywords Review+ added; Review? removed
  • Resolution wontfix deleted

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 5 years ago by tobiasz.cudnik

  • Status changed from reopened to closed
  • Resolution set to fixed

Fixed with [4177].

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