Opened 16 years ago
Closed 16 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 16 years ago by
| Owner: | set to Tobiasz Cudnik |
|---|---|
| Status: | new → assigned |
comment:2 Changed 16 years ago by
| Keywords: | IE added |
|---|
Changed 16 years ago by
| Attachment: | 3946.patch added |
|---|
comment:3 Changed 16 years ago by
| 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 16 years ago by
| Keywords: | Review- added; IE Review? removed |
|---|
IE is not the only browser affected. The patch doesn't work in Firefox.
Changed 16 years ago by
| Attachment: | 3946_2.patch added |
|---|
comment:5 Changed 16 years ago by
| Keywords: | Review? added; Review- removed |
|---|
Last minute fix broken focus event in FF. Reverting missing code.
comment:6 Changed 16 years ago by
| Keywords: | Review+ added; Review? removed |
|---|
Note: See
TracTickets for help on using
tickets.
Two focus handlers aren't fired in IE6 strict. Those one which un-block blur lock.