FF:cannot format entire table column - adds code outside <td></td> tags of 1st cell

[Tanya]

To replicate the bug, create a table with more than one row, enter text into cells then select content of an entire column or row. Select a formatting option such as BOLD or Right Align. Result: firstly, the entire column is not affected secondly, the first cell stuffs up. In the right align example (shown below) the first <td> is altered, placing two empty <p> tags and css styles. In the editor it looks like another cell has been added to the table.

I get a similar issue when trying to format the whole table as "normal" paragraph text.

<table border="1" cellpadding="0" cellspacing="0">

<tbody>

<tr>

<p style="text-align: right;">&nbsp;</p> <td style="text-align: right;">aaaaa</td> <p style="text-align: right;">&nbsp;</p> <td>aaa</td>

</tr> <tr>

<td>aa</td> <td>aa</td>

</tr>

</tbody>

</table>

[Wojciech Olchawa]

Confirmed in FF. Works fine with IE.

This bug is a little bit more complicated. Following your example I will write another step by step instruction to reproduce this bug.

1. Create a 3 row table with th sample code below:

<table width="200" cellspacing="1" cellpadding="1" border="1">
    <tbody>
        <tr>
            <td>cell1</td>
            <td>cell2</td>
        </tr>
        <tr>
            <td>cell3</td>
            <td>cell4</td>
        </tr>
        <tr>
            <td>cell5</td>
            <td>cell6</td>
        </tr>
    </tbody>
</table>

2. Highlight the first row and press Right Justify button.

3. The result is:

<table width="200" cellspacing="1" cellpadding="1" border="1">
    <tbody>
        <tr>
            <p style="text-align: right;">&nbsp;</p>
            <td style="text-align: right;">cell1</td>
            <p style="text-align: right;">&nbsp;</p>
            <td>cell2</td>
        </tr>
        <tr>
            <td>cell3</td>
            <td>cell4</td>
        </tr>
        <tr>
            <td>cell5</td>
            <td>cell6</td>
        </tr>
    </tbody>
</table>

The expected behavior would be to have a code like this:

<table cellspacing="1" cellpadding="1" width="200" border="1">
    <tbody>
        <tr>
            <td style="text-align: right">cell1</td>
            <td style="text-align: right">cell2</td>
        </tr>
        <tr>
            <td>cell3</td>
            <td>cell4</td>
        </tr>
        <tr>
            <td>cell5</td>
            <td>cell6</td>
        </tr>
    </tbody>
</table>

So Firefox messes up the table. I've added to screenshots to show the results.

It is possible that we have a ticket opened for this bug. I will look it out.

[Wojciech Olchawa]

#1758 has been marked as DUP

[Karen Ananiev]

I'm not sure that it is good solution but it works :)

[Karen Ananiev]

It seems that CreateBookmark creates <span>s between <td>s when some cells are selected.

[jorenm@…]

I don't know if that patch works or not, because I do not know how to apply it. It's still broken in the newest version of FCK though, so this is the solution that fixed it for me:

// Check if there are selected cells, if there are then move the selection 
// inside the TDs so the following while loop works properly
        
// Don't create a bookmark in firefox as it won't reselect the table cells as 
// they were before, and firefox doesn't lose the selection in the iframe anyway.
var tds = FCKTableHandler.GetSelectedCells();
if (tds && FCKBrowserInfo.IsGecko) {
    range.SetStart(tds[0], 1);
    range.SetEnd(tds.last(), 1);
} else {
// Store a bookmark of the selection since the paragraph iterator might
// change the DOM tree and break selections.
    var bookmark = range.CreateBookmark();
}

And then make sure bookmark exists.

if (bookmark) {
    range.MoveToBookmark( bookmark ) ;
    range.Select() ;
}      

[Jakub Ś]

No longer reproducible in CKEditor.