resizing handles of dragresizetable plugin are created in the fckeditor.html frame

[Alfonso Martínez de Lizarrondo]

Due to ​https://bugzilla.mozilla.org/show_bug.cgi?id=130078 that means that the created elements are behind the editing div and aren't visible if we are running in chrome: (write area)

The solution seems to be to use the FCK.EditorDocument to create and position those elements.

[Alfonso Martínez de Lizarrondo]

proposal

[Alfonso Martínez de Lizarrondo]

The proposed patch just factorizes some calls in a way that it's possible to change the behavior of the plugin easily to create the divs in the edited frame. in fact the AfterSetHTML function could be removed and _avoidStyles could be left as an empty function.

Later I just have to call

FCK.Events.AttachEvent( "OnAfterSetHTML", FCKDragTableHandler.AfterSetHTML ) ;
FCKDragTableHandler._GetIframeOffset = function() { return {x:0, y:0}; };
FCKDragTableHandler._GetTablePosition = function( w, table) { return FCKTools.GetDocumentPosition(w, table ) ; };

and that way the code will work only on the edited frame.

The drawback is that the created divs can be affected by the page styles, so doing this for the general core could be troublesome, but just the suggested changes (minus the .AfterSetHTML and leaving _avoidStyles as a dummy will make it possible to include the plugin in Write Area easily.

Thoughts? other approaches?

[Martin Kou]

The plugin works ok after the patch, but it seems to me the "AfterSetHTML()" function added in the patch is not being called?

[Alfonso Martínez de Lizarrondo]

updated patch

[Alfonso Martínez de Lizarrondo]

You're right.

That function isn't needed in this code, it should be created by the external code that will adjust the other functions.

[Alfonso Martínez de Lizarrondo]

Fixed with [1365]