18839 - toggling style.display = 'none'/'table-row' works only once

Status: VERIFIED WORKSFORME

(Core :: Layout: Tables, defect, P3)

Description

[martin.honnen]

I have some code which on button click sets the
.style.display
value of even rows in a table alternatively to
'table-row'/'none'
Setting to 'none' and back to 'table-row' is visually executed once afterwards
the screen display of the rows remains visible though the javascript/dom value
gets set to 'none'.
To check the bug press the button three times. The third press has not the
desired effect.


< SCRIPT> var visible = true;
function toggleRows (tid) {
  var table =
document.getElementById(tid);
  for (var r = 1; r < table.rows.length; r += 2)

   table.rows[r].style.display = visible ? 'none' : (navigator.appName ==
'Microsoft Internet Explorer') ? 'inline' : 'table-row';
  visible = !visible;
}

< /SCRIPT>< /HEAD>< BODY>< BUTTON ONCLICK="toggleRows('aTable');"> toggle rows< /BUTTON
>< BUTTON
ONCLICK="alert(document.getElementById('aTable').rows[1].style.display);"> show
rows[1].style.display
< /BUTTON>< BR>< TABLE ID="aTable">< TR BGCOLOR="lime"><
TD> Kibology
< /TD>< TD> for
< /TD>< TD> all
< /TD>< /TR>< TR BGCOLOR="yellow">< TD
> All
< /TD>< TD> for
< /TD>< TD> Kibology
< /TD>< /TR>< TR BGCOLOR="lime">< TD> Kibology

< /TD>< TD>

Comment 1

[martin.honnen]

Attachment: bug demo

Comment 2

[vidur (gone)]

In an attempt to get my bug list in order again, marking all the bugs I have
currently as ASSIGNED.

Comment 3

[troy]

This is a layout bug and not a DOM bug so changing component

Comment 4

[troy]

However, the page seems to be working very well for me with today's build so I'm
marking this WORKSFORME

Comment 5

[Amarendra Hanumanula]

 The given page with .style.display works fine for me on build ID : 2001122603
Marking verified

Comment 6

[Pulsebot]

Pushed by ffxbld-merge:
https://hg.mozilla.org/releases/mozilla-beta/rev/e6005c251469
[fenix] For https://github.com/mozilla-mobile/fenix/issues/8984: Added color for "Share" menu's "Recently Used" in Dark theme (https://github.com/mozilla-mobile/fenix/pull/18839)