Opera : We can't open more than one sub menu

[Frederico Caldeira Knabben]

With Opera:

1. Open CKEditor and insert a table.

2. Open the context menu and keep your mouse over Cell option and see that Cell Sub menu opens.

3. Now keep your mouse over Row option.

Expected Result: Row sub menu opens.

Actual Result: Row sub menu does not open and context menu closes & focus goes back to table in editor body.

[Frederico Caldeira Knabben]

This one is new to Opera 11. It used to work with Opera 10.50.

[Hallvord R. M. Steen (Opera Software)]

just tried it with 11.10 and the table in the little red riding hood text on ckeditor.com/demo and it seemed to work fine there. I will try the SVN trunk version shortly, but can anybody reproduce the failure on the normal demo page?

[Hallvord R. M. Steen (Opera Software)]

Just tested this with 11.10 and the freshly updated SVN trunk, still works for me..?

[Hallvord R. M. Steen (Opera Software)]

Those who can reproduce this problem: any errors in the error console?

[Frederico Caldeira Knabben]

I can constantly reproduce the problem with both the trunk and the online demo.

I'm testing it with "Opera/9.80 (Windows NT 6.1; U; en) Presto/2.8.131 Version/11.11". No errors or warnings on console.

[Wiktor Walc]

Same problem on my side, the "Cells" submenu opens, but the context menu disappears when moving the mouse down, over the "Rows" item (I've checked it on the "Little Red Riding Hood" page).

Version 11.11
Build 2109
Platform Win32
System Windows 7
Browser identification Opera/9.80 (Windows NT 6.1; U; en) Presto/2.8.131 Version/11.11

One more observation: when opening the context menu for the second time, it disappears straight after moving the mouse over the "Cells" item.

[Hallvord R. M. Steen (Opera Software)]

11.11 seems to fire unexpected blur events - looking at this now..

[Hallvord R. M. Steen (Opera Software)]

Looking at floatpanel/plugin.js

First, a small optimisation: this part of the if clause inside the onblur code:

if ( CKEDITOR.env.ie && !this.allowBlur() ||

is superfluous because the function starts with a !this.allowBlur() check.

The real problem is that Opera is firing a number of unexpected blur events on the document inside the IFRAME where the menu is. I'm still not sure why this happens, I need to get a better overview of what exactly the editor does when (re)creating the menu to try to figure out which part of it makes Opera fire those blur event we did not previously fire..

[Hallvord R. M. Steen (Opera Software)]

Garry, if you do that it's not possible to close right-click- and sub-menus.. Right?

Any idea what exactly triggers the problem..?

[Garry Yao]

My fault, the patch doesn't really address the issue.

[Frederico Caldeira Knabben]

Replying to hallvord@…:

Looking at floatpanel/plugin.js

First, a small optimisation: this part of the if clause inside the onblur code:

if ( CKEDITOR.env.ie && !this.allowBlur() ||

is superfluous because the function starts with a !this.allowBlur() check.

Correct. Fixed with [7037].

[Hallvord R. M. Steen (Opera Software)]

With some developer help, I've identified the reason for this regression: it is triggered by the overflow:hidden style for the

.cke_skin_kama .cke_panel

selector in panels.css

Is it possible to remove overflow:hidden without running into issues with other browsers?

[Hallvord R. M. Steen (Opera Software)]

(This regression only affects 11.11 by the way, 11.50 will work fine again).

I guess you could also detect Opera and set style.overflow='visible' on the IFRAME's parent element.

[Hallvord R. M. Steen (Opera Software)]

(If you go the browser sniffing route - a bit complex to detect this problem in advance - I suggest setting style.overflow only in Opera 11.11 - it might help debugging future issues if we limit unexpectedness as much as possible.)

I've reported this internally as DSK-339087 and will add a test or two to make sure we don't bring back this regression in the future.

[Garry Yao]

The proposed patch is enough to eliminate this bug, it also simplifies some messy code a bit, I've also tried with hallvord's proposal(removing the overflow style) - unbelievable but it works also.

[Garry Yao]

10.5 has got the massive focus event bug fixed, so the patch is not needed anymore, thanks for all participants on it.

[Satya Minnekanti]

Replying to garry.yao:

10.5 has got the massive focus event bug fixed, so the patch is not needed anymore, thanks for all participants on it.

@garry.yao I could still reproduce this bug on latest Opera 11.11. Could you please explain me why you closed this ticket?

[Hallvord R. M. Steen (Opera Software)]

He probably meant that 11.50 has got the bug fix that makes menus work again :)

[Satya Minnekanti]

Replying to hallvord@…:

He probably meant that 11.50 has got the bug fix that makes menus work again :)

@hallvord then he should not close the ticket unless we verify that it is fixed in 11.50. Could you please re open the ticket

[Hallvord R. M. Steen (Opera Software)]

Sure, I can reopen if you wish.

I intend to ship a site patch here - a modification to Opera's "compatibility hacks" (browser.js) that will be downloaded by all Opera 11.1x instances automatically and applied anytime Opera sees the script file name 'ckeditor.js'. This means that within a couple of weeks (vacation time will delay the next browser.js release unless something unexpected happens) the menus should start working in 11.11 as well. I think this is nicer than complicating CKEditor's code for a legacy bug (although in this case the code changes might actually be a simplification ;) Haven't studied the proposed patch in detail. )

[Hallvord R. M. Steen (Opera Software)]

I have reported PATCH-461 "ship a script-specific patch for DSK-339087 (unexpected blur events breaks CKEditor menus in 11.11 (CK #7882))" to track sitepatching. The patch's launch will be announced on ​http://my.opera.com/sitepatching/blog/ .

[Satya Minnekanti]

Fixed in Opera 11.50. please close this ticket.

[Garry Yao]

Replying to hallvord@…:

He probably meant that 11.50 has got the bug fix that makes menus work again :)

My typo, I did mean 11.5.