44148 - Adding and removing textfield to/from table cell breaks cell borders

Status: RESOLVED WORKSFORME

(Core :: Layout, defect, P3)

Description

[Taras Tielkes]

Using HTML DOM1 to programmaticaly add, and later remove a textfield from a
table cell shows that the table borders are (unnecessarily?) repainted is a
wrong way.

Most of the times the RIGHT cell border is moved a pixel, or a couple of pixels
to the right direction. This seems not the reaction to a reflow, as the
erroneous repaint occurs when you REMOVE the textfield, not when you add it.

Also, the textfield is much smaller that the table cell.

Behaviour can be seen in all the latest builds on Windows, and sometimes also
occurs when you select a part of a table, and then remove your selection.

I will attach a testcase that shows this behaviour. The testcase has a small
HTML table, and two form buttons. Press the first button to add a textfield to
the table cell, then press the second button to remove it again. This behaviour
can be observer with both strict DTDs as without. Using a strict DTD and setting
the environment variable ENABLE_STRICT=true makes no difference.

Comment 1

[Taras Tielkes]

Attachment: Testcase that shows described layout bug with table cells and input fields

Comment 2

[Kathleen :Brade]

I'm guessing this is a table layout bug for karnaze; cc rods if karnaze is out.

Comment 3

[Taras Tielkes]

Also seen on Linux, changing plaf/OS to All/All.

Comment 4

[karnaze (gone)]

Not enough of the table area is being invalidated. Accepting the bug.

Comment 5

[Taras Tielkes]

This is a pretty serious layout bug.

Any chance this one can get a higher priority (nsbeta3?) tag?

Comment 6

[Taras Tielkes]

PDT: this sample uses basic DOM1 to insert a textfield in a table. When removing
the textfield, almost all tables break.

Comment 7

[Taras Tielkes]

[e-mail] , I'm having a lot of issues with this one. When you have
time to take a look, I'll be willing to provide a lot of testing and QA. Give a
yell if I can help.

Comment 8

[karnaze (gone)]

I'm downgrading this to nsbeta3- because it only seems to fail with collapsing 
borders (CSS2). 

Comment 9

[timeless]

adding brackets, although i disagree w/ the status.

Comment 10

[karnaze (gone)]

Moving m1.0.

Comment 11

[Taras Tielkes]

Looks fine in a 4/4/2K1 build.

Don't know if it really depends on 41262, but it seems to work now.
Shall I mark WORKSFORME?

Comment 12

[karnaze (gone)]

Marking worksforme based on comments.