224517 - using style.display=block/none repeatedly on a tr breaks table

Status: RESOLVED DUPLICATE of bug 97506

(Core :: Layout: Tables, defect)

Description

[paul stanton]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.5) Gecko/20031007 Firebird/0.7
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.5) Gecko/20031007 Firebird/0.7

using style.display=block/none repeatedly on a tr creates extra padding for row
and mis-alignment of columns

Reproducible: Always

Steps to Reproduce:
1. visit http://wanto.f2o.org/****/trDisplay.html
2. repeatedly click button 'toggle'

Actual Results:  
the row containing 'i am sam' shows and hides it's self, but with extra padding
and mis-alignment

Expected Results:  
the row containing 'i am sam' shows and hides it's self

Comment 1

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

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

Comment 2

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

The default value of the 'display' property for table rows is 'table-row', not
'block', and except for the extra space being introduced below our behavior is
correct.  You're better off setting it to "" instead of "block" -- that should
work across browsers.

The valid issue is a duplicate.

Comment 3

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

*** This bug has been marked as a duplicate of 97506 ***

Comment 4

[paul stanton]

yes; using "" instead of "block" works for mozilla, as does "table-row",
however, neither of these work in ie 6. not that this is mozilla-dev's problem,
just a note for developers reading this, in order to achieve cross browser
support, i will have to write detection code. :~(

Comment 5

[Mats Palmgren (inactive)]

Just out of curiosity, does this work in IE:

<style>
.a { display:none; }
.b { display:block; display:table-row; }
</style>

document.getElementById("this_one").setAttribute('class',
  document.getElementById("this_one").getAttribute('class') == "a" ? "b" : "a");

Comment 6

[paul stanton]

no