#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

Reply via email to