72303 - Table cell borders cannot be changed dynamically

Status: RESOLVED FIXED

(Core :: DOM: CSS Object Model, defect)

Description

[Erik Arvidsson]

The style for borders of table cells (only td tested) cannot be changed 
dynamically.

Neither setting the style.border* or changing the className (to change the 
border) updates the visual state of the border for table cells.

This works fine for block and inline elements.

Even if this is just a visual bug I think it is a rather serious bug because it 
is a commonly used task.

Comment 1

[Erik Arvidsson]

Attachment: Press mouse button on any of the cells and notice that the border does not change

Comment 2

[Johnny Stenback (:jst)]

Table reflow/painting problem, over to karnaze.

Comment 3

[karnaze (gone)]

The testcase works on the 1st cell of each row but not the 2nd cell. I changed 
the testcase to have only one row and in the 2nd cell to not do the onmouseup. 
Mousing on the 2nd cell still did not show the correct border until I minimized 
and maximized Viewer. Since, the table cell frame is invalidating its entire 
rect during this kind of reflow, I am suspecting a rendering problem. 
Reassigning to kmcclusk.

Comment 4

[karnaze (gone)]

*** Bug 70756 has been marked as a duplicate of this bug. ***

Comment 5

[karnaze (gone)]

From bug 70756 (in case this really isn't a dup) - "Even though nsTableCellFrame 
is invalidating its entire rect, when it gets painted, the dirty rect is not 
right."

Comment 6

[Kevin McCluskey (gone)]

Most of the borders change color when I hold down over them. Portions of some of
the borders do not change color. This is most likely a twips->pixel roundoff
error when invalidating the bounds for the frame. All frames (x,y) offset must
be aligned to pixel boundaries.

Comment 7

[Kevin McCluskey (gone)]

It is working correctly in todays build. My change to text frame invalidation
may have fixed the problem. Marking bug FIXED.