Ticket #3946 (closed Bug: fixed)

Opened 5 years ago

Last modified 5 years ago

Unable to hide contextmenu

Reported by: garry.yao Owned by: tobiasz.cudnik
Priority: Normal Milestone: CKEditor 3.0
Component: UI : Context Menu Version:
Keywords: Confirmed Review+ Cc:

Description

Reproducing Procedures

  1. Open a context menu with a second-level sub menu;
  2. Activate the sub menu by mouse over the menu item;
  3. Now mouse move back to the first-level menu;
  4. Activate the same sub menu again;
  5. Click on the document text;
    • Actual Result: The context menu stays visible.

Attachments

3946.patch (1.2 KB) - added by tobiasz.cudnik 5 years ago.
3946_2.patch (1.9 KB) - added by tobiasz.cudnik 5 years ago.

Change History

comment:1 Changed 5 years ago by tobiasz.cudnik

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

comment:2 Changed 5 years ago by tobiasz.cudnik

  • Keywords IE added

Two focus handlers aren't fired in IE6 strict. Those one which un-block blur lock.

Changed 5 years ago by tobiasz.cudnik

comment:3 Changed 5 years ago by tobiasz.cudnik

  • Keywords Review? added

Issue was caused by not-working forced iframe-focus logic for IE non-quirks. This patch offers also optimization for displaying same child menu (as visible), which are now skipped. This prevents flickering issue on IE and generally reduces executed code.

comment:4 Changed 5 years ago by garry.yao

  • Keywords Review- added; IE Review? removed

IE is not the only browser affected. The patch doesn't work in Firefox.

Changed 5 years ago by tobiasz.cudnik

comment:5 Changed 5 years ago by tobiasz.cudnik

  • Keywords Review? added; Review- removed

Last minute fix broken focus event in FF. Reverting missing code.

comment:6 Changed 5 years ago by fredck

  • Keywords Review+ added; Review? removed

comment:7 Changed 5 years ago by tobiasz.cudnik

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

Fixed with [3908].

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