50794 - table cell bg color doesn't work the first time.

Status: VERIFIED FIXED

(Core :: DOM: Editor, defect, P3)

Description

[sujay]

using 8/30 build of netscape

1) launch netscape
2) launch composer
3) insert default table
4) click in one of the cells
5) pull down Table | Tableor cell background color
6) pick a colore
7) click OK

notice you don't see the table cell color change...

it somehow works the second or third time...



all platforms.

Comment 1

[rubydoo123]

actually, the color is visible when you type a character, something needs to 
trigger the cell to change and the character entry does that.

moving this out to future

Comment 2

[Charles Manske]

This isn't the only circumstance when changing a cell or color table color
doesn't trigger repaint. It sucks!
Maybe we can address this issue when we do "performance" work, which would
seem to involve examining the incremental painting code?
CCing Marc, since he is expert with similar problems in the past.

Comment 3

[rubydoo123]

I agree this is awful, I will reset to perf work and set it to m19

Comment 4

[Marc Attinasi]

The color of the cell is changing immediatly for me on today's build - did it 
get fixed, or is it tempermental? I tried both menu choices:
Table | Table or Cell Background Colors... and 
Format | Table Cell Properties

Comment 5

[rubydoo123]

Marc, it's still there, try this
1.insert default table
2.insert 4-5 lines of text in upper left cell
3.select all text and align right
4.set caret to the left of the first character in the first line
5.select a background color

only part of the cell fills, you must do something at this point for the rest of 
the cell to fill, it happens in other cases too -- this is just the easist to 
dup

Comment 6

[rubydoo123]

reviewed by Bijal and beppe, setting to future

Comment 7

[verah (gone)]

Is this a duplicate of 53741? Both have the relnote3 keyword.

Comment 8

[Kristi Fredrickson]

cc'ng myself

Comment 9

[kinmoz]

This seems to be fixed already. I tried sujay's original problem, and beppe's 
problem, and both seem to work ok now.

Comment 10

[sujay]

verified in 4/19 build.

Comment 11

[Kristi Fredrickson]

Jatin -- another one that was release noted that, if it's still in that doc, will 
need to come out when you do the next draft