#2438: tablehandler not operating on proper row - Context Menu js error
-------------------------------+--------------------------------------------
Reporter: boorad | Owner:
Type: Bug | Status: new
Priority: Normal | Milestone:
Component: UI : Context Menu | Version: FCKeditor 2.6
Keywords: |
-------------------------------+--------------------------------------------
It seems that in a <table> we are seeing the right-click context menu pop
up for all rows except the last. Firebug is giving us an error on this
line:
http://dev.fckeditor.net/browser/FCKeditor/tags/2.6/editor/_source/internals/fcktablehandler.js#L274
Following the call stack down to
http://dev.fckeditor.net/browser/FCKeditor/tags/2.6/editor/_source/internals/fcktablehandler.js#L302
got me wondering if that line should be
{{{
var rowIdx = cells[0].parentNode.rowIndex - 1;
}}}
as rowIndex doesn't seem to be zero-based. This solved our problem.
A few notes:
* /trunk is the same, no changes have been made since release 2.6.
* I have only tested this fix on FF3 / OS X, and I'm not sure how it
reacts with other browsers or OSes.
* I'm not sure of other ramifications of this change if it's made.
* If this issue is valid, where else does it need to be changed? I'll
leave that as an exercise for the author. :)
--
Ticket URL: <http://dev.fckeditor.net/ticket/2438>
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
