Opened 8 years ago
Closed 8 years ago
#3946 closed Bug (fixed)
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
- Open a context menu with a second-level sub menu;
- Activate the sub menu by mouse over the menu item;
- Now mouse move back to the first-level menu;
- Activate the same sub menu again;
- Click on the document text;
- Actual Result: The context menu stays visible.
Attachments (2)
Change History (9)
comment:1 Changed 8 years ago by tobiasz.cudnik
- Owner set to tobiasz.cudnik
- Status changed from new to assigned
comment:2 Changed 8 years ago by tobiasz.cudnik
- Keywords IE added
Changed 8 years ago by tobiasz.cudnik
comment:3 Changed 8 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 8 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 8 years ago by tobiasz.cudnik
comment:5 Changed 8 years ago by tobiasz.cudnik
- Keywords Review? added; Review- removed
Last minute fix broken focus event in FF. Reverting missing code.
comment:6 Changed 8 years ago by fredck
- Keywords Review+ added; Review? removed
comment:7 Changed 8 years ago by tobiasz.cudnik
- Resolution set to fixed
- Status changed from assigned to closed
Fixed with [3908].
Note: See
TracTickets for help on using
tickets.
Two focus handlers aren't fired in IE6 strict. Those one which un-block blur lock.