102930 - When using split cell, the cells formed in the split can take the background color of cells in other columns.

Status: VERIFIED FIXED

(SeaMonkey :: Composer, defect)

Description

[TucsonTester2]

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:0.9.4)
Gecko/20011002 Netscape6/6.2
BuildID:    20011002

This problem occurs when you form a cell by joining cells from more than one row
and more than one column.  If you split the cell, then the cells formed by the
split that are in the top row will retain the background color that the cell had
before it was split.  But cells formed in any row other than the first will take
on the background color of a cell in an adjacent row that was not involved in
the split.

Reproducible: Always
Steps to Reproduce:
1.Open a new page in composer
2.Click on table, create a table with 2 rows and 3 columns
3.Double click on a cell in the first column
4.In cell properties, change selection to column and change background color to
black.
5.Double click a cell in the second column, select column and change background
color to red
6.Double click a cell in the third column, select column and change background
color to blue
7.While holding down the cntrl key, click on the two cells in the second column
and the two cells in the third column so they are selected
8.Right click on one of the selected cells and click Join selected cells.
9.Right click on the large cell created by joining, and selced split cell.

Actual Results:  The first row has cells colored black, red, red.  The second
row has cells colored black, black, black.

Expected Results:  I would expect all cells formed by splitting to retain the
same color they had before they were split.  I would not expect composer to
arbitrarily decide to assign a color being used by a cell that was not part of
the split.

When splitting cells formed from joining multiple rows and columns, the cells in
the first row will always retain the color they had before splitting.  Cells in
any other row will take their color from the cell in their row that is to the
left of the cell that is being split.  Unless the cell to the left is one that
was formed by joining other cells.  In that case, the cells formed by the split
will take their color from the closest cell to the right in that row that is not
part of a cell formed by joining multiple cells.

Comment 1

[Julien Cegarra]

Confirmed. (BuildID: 2001100903)

To understand the steps you have to change background properties for <B>both</B>
cell of each column (steps 4, 5 & 6).

Comment 2

[Julien Cegarra]

It seems for me its a part of a larger bug :
Bug#102930 (split dont remember color)
Bug#100766 (split dont remember size)
Bug#102967 (split crash)

Comment 3

[Charles Manske]

Easy to fix.
Julien: Those bugs you listed are not related.

Comment 4

[Charles Manske]

Attachment: Fix. Copy original cell's background color to newly created cells during split

Comment 5

[Kathleen :Brade]

Comment on attachment 53878 [details] [diff] [review]
Fix. Copy original cell's background color to newly created cells during split

r=brade

Comment 6

[Joe Hewitt (gone)]

sr=hewitt

Comment 7

[Joe Hewitt (gone)]

Comment on attachment 53878 [details] [diff] [review]
Fix. Copy original cell's background color to newly created cells during split

darn, keep forgetting to use this thing ;)

Comment 8

[Charles Manske]

checked in

Comment 9

[michael wendell]

Verified fixed on 20011022 trunk.