#1917: mergedown error / three cell
----------------------+-----------------------------------------------------
Reporter: aceseol | Owner: martinkou
Type: Bug | Status: assigned
Priority: Normal | Milestone: FCKeditor 2.6.3
Component: General | Version: SVN
Resolution: | Keywords: Confirmed IE HasPatch Review?
----------------------+-----------------------------------------------------
Changes (by martinkou):
* keywords: Confirmed IE HasPatch => Confirmed IE HasPatch Review?
Comment:
Lines 733 and 734 from Karena's patch isn't really needed because we're
counting the rowSpan from the tableMap data structure, so we don't really
have to add up the rowSpan of the current cell if we've already stored
the sum in another variable.
Karena's patch has worked around an IE bug here though, which is the
essential component of the fix. Our current code does not work in IE
because we're depending on IE reporting the correct rowSpan for the td
elements - unfortunately when a td is not attached to a tr IE would always
report its rowSpan to be 1. So Karena's way of storing the sum in a
variable works around the problem.
Proposing a new patch with lines 733 and 734 from Karena's patch removed.
--
Ticket URL: <http://dev.fckeditor.net/ticket/1917#comment:9>
FCKeditor <http://www.fckeditor.net>
The text editor for Internet
-------------------------------------------------------------------------
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://sourceforge.net/services/buy/index.php
_______________________________________________
FCKeditor-Trac mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/fckeditor-trac