#1865: Contextmenu in last row of a table with thead doesn't work
--------------------------------+-------------------------------------------
Reporter: kwillems | Owner: alfonsoml
Type: Bug | Status: assigned
Priority: Normal | Milestone: FCKeditor 2.6.4
Component: UI : Context Menu | Version: FCKeditor 2.5.1
Resolution: | Keywords: Confirmed
--------------------------------+-------------------------------------------
Changes (by alfonsoml):
* keywords: Confirmed HasPatch => Confirmed
Comment:
Regarding the 1865_1.patch:
In FCKTableHandler.HorizontalSplitCell it tries to get a reference to the
table starting from the cell and reading .offsetParent but
[https://developer.mozilla.org/En/DOM/Element.offsetParent MDC] states
that the element returned should be the first positioned ancestor. The
fact that it returns the table might not be crossbrowser compatible so it
should be better to move up using the DOM: FCKTools.GetElementAscensor(
cell, 'table' )
The modifications in FCKTableHandler._GetCellIndexSpan are wrong.
It might be better to provide a patch that does a single thing as it will
be easier to understand and review.
--
Ticket URL: <http://dev.fckeditor.net/ticket/1865#comment:13>
FCKeditor <http://www.fckeditor.net/>
The text editor for Internet
-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
_______________________________________________
FCKeditor-Trac mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/fckeditor-trac
