#2389: Merge Cells removes attributes from tr in table
-----------------------+----------------------------------------------------
Reporter: mattleff | Owner:
Type: Bug | Status: new
Priority: Normal | Milestone:
Component: General | Version: FCKeditor 2.5 Beta
Resolution: | Keywords: Confirmed DataLoss
-----------------------+----------------------------------------------------
Comment(by alfonsoml):
Having a function named !CopyAttributesToArray that returns an object
looks really strange. The comments should reflect the functionality in a
generic way (don't use "on the row element", or that "could probably be
used in other scenarios")
To me it looks strange that the functions added to fckdomtools are
asymetric. One takes care of doing all the loop, but the second one only
applies one attribute at a time.
The logic about rowCount is flawed. You can test that merging some cells
in this table changes the classes:
{{{
<table cellspacing="1" cellpadding="1" border="1" width="200">
<tbody>
<tr class="odd">
<td> </td>
<td> </td>
</tr>
<tr>
<td> </td>
<td> </td>
</tr>
<tr class="odd">
<td> </td>
<td> </td>
</tr>
</tbody>
</table>
}}}
--
Ticket URL: <http://dev.fckeditor.net/ticket/2389#comment:4>
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
