[IE7] Table context menu submenus doesn't appear properly.

[w.olchawa]

Steps to reproduce:

1. Go to our demo page  http://www.fckeditor.net/demo

2. Insert a table.

3. Right-click inside a cell to activate the context menu

4. Go to a sub-menu e.g. Cell, Row or Column

Notice that the submenu appears not besides the context menu but practically on it.

Expected behavior would be displaying a submenu besides the original context menu.

Tested using 2.6 beta and SVN version on IE7. It works fine in IE6 and FF2.

[fredck]

This is not a regression. I've confirmed it with version 2.4.3, so it's not a regression.

Even if annoying, it is not a big issue, fortunately.

[alfonsoml]

Proposed patch

[alfonsoml]

I've managed to create a patch that fixes the problem with almost no drawbacks.

The problem is that the submenus are created having the main menu as its parent window, so their dimensions are restricted by that main menu. I've coded a little logic that takes care to increase the size of that panel and so it allows the submenu to have whatever dimensions are needed.

I'm checking now (I wanted to do it earlier but I forgot) that the behaviour with the Hebrew language isn't too nice. I'll have to work on it before requesting review (anyway, if someone has any idea I'll be glad)

[alfonsoml]

Patch including RTL handling.

[alfonsoml]

Ok, I needed to rest a little to find the solution but now it works fine in all my tests.

The only drawback of the patch is that there's a white area behind the submenu, but I think that it's a very minor thing comparing to how hard is currently to access the submenus in IE7 sometimes (for example if you are at the right side of the screen).

[alfonsoml]

Fixed with [2386]

[AlfK]

Case reopened due to error described in:  http://www.fckeditor.net/forums/viewtopic.php?f=5&t=10949&p=28840#p28840

[alfonsoml]

I have no plans to work on three levels menus as I haven't seen any patch to add such menus or any plugin that uses them.

Maybe someone else wants to spend their time fixing that problem. Clearing out the review so it doesn't show in the default queries also.

[martinkou]

It took me a while to figure out what Alfonso's patch was doing, but turns out the modifications needed aren't big.

[martinkou]

Modified Alfonso's patch to support more than 2 submenu levels.

[fredck]

The patch looks good. There is just one small detail. Please cleanup the this.RelativeElement property, because it could easily result on memory leak.

[martinkou]

Added the cleanup code.

[martinkou]

Fixed with [2597].

Click here for more info about our SVN system.