Closed
Bug 21143
Opened 25 years ago
Closed 25 years ago
Rows/columns that are collapsed and then shown are screwed up
Categories
(Core :: Layout: Tables, defect, P3)
Tracking
()
VERIFIED
FIXED
People
(Reporter: hyatt, Assigned: karnaze)
References
Details
Here's an important bug that is biting me over in the tree widget, Chris. If
you set the visibility of a column or row to collapse, it works just fine. if
you later set the visibility back to normal, the contents of the cells no longer
display.
Here is a simple HTML test case that shows the problem.
<table cellspacing=0 border=1 style="background-color:orange;">
<caption><b>after</b></caption>
<colgroup>
<col>
<col id="foo">
<col>
<col>
</colgroup>
<tr><td>C11<td>C12<td>C13<td>C14
<tr style="visibility:collapse;"><td>C21<td rowspan=2 colspan=2>C12 C13 C22
C23<td>C14
<tr><td>C31<td>C34
<tr><td>C41<td>C42<td>C43<td>C44
</table>
<button onclick="var v = document.getElementById('foo');
var s = v.getAttribute('style');
if (s != '')
v.removeAttribute('style');
else v.setAttribute('style', 'visibility: collapse')">
Click me to toggle the second column!</button>
In the above test case, the first time you click the button, the column
will collapse. The second time you click it, the column will re-appear, but
the cells will all be empty.
This also happens with row collapsing. This is a blocker tree widget bug,
since i need to be able to do dynamic row/column effects.
chrisd - when this is marked fixed, can you let me know? I'd like to verify the
specific case in the duplicate bug for mail. Thanks.
Assignee | ||
Updated•25 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Comment 3•25 years ago
|
||
I have already tried it in mail which is why I marked it a duplicate.
Assignee | ||
Updated•25 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 4•25 years ago
|
||
Fixed with latest table checkin.
Comment 5•25 years ago
|
||
Using 7/13 build, columns collapse and reappear appropriately. Verifying bug
fixed.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•