Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#3390 closed Bug (fixed)

[FF] Open submenu

Reported by: Garry Yao Owned by: Garry Yao
Priority: Normal Milestone: CKEditor 3.0
Component: General Version:
Keywords: Confirmed Firefox Review+ Cc:

Description

Reproducing Procedures

  1. Open the replace by class example page;
  2. Insert a table, right click to open context menu which has sub-menu;
  3. Navigate to the menu item which would trigger sub-menu, press 'arrow-right' key.
    • Expected Result: Sub menu show up.
    • Actual Result: Javascript error thrown.

Attachments (1)

3390.patch (462 bytes) - added by Garry Yao 10 years ago.

Download all attachments as: .zip

Change History (7)

Changed 10 years ago by Garry Yao

Attachment: 3390.patch added

comment:1 Changed 10 years ago by Garry Yao

Owner: set to Garry Yao
Status: newassigned

None-IE doesn't support click on link element, change to invoke handler directly.

comment:2 Changed 10 years ago by Garry Yao

Keywords: Review? added

comment:3 Changed 10 years ago by Martin Kou

Keywords: Review+ added; Review? removed

comment:4 Changed 10 years ago by Frederico Caldeira Knabben

There is still no attention to the coding style. Please change it to the following when committing:

if ( link ) 
	link.$.click ? link.$.click() : link.$.onclick();

comment:5 Changed 10 years ago by Garry Yao

Resolution: fixed
Status: assignedclosed

Fixed with [3404] and [3411].

comment:6 Changed 10 years ago by Frederico Caldeira Knabben

Attention to details: [3414]

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