Opened 5 years ago

Closed 5 years ago

#1485433 closed Bugs (fixed)

validator complains for embedded script

Reported by: tensor Owned by:
Priority: 5 Milestone: 0.2-stable
Component: Client Scripts Version: 0.2-beta
Severity: trivial Keywords:
Cc:

Description

Advanced options checkboxs is a nice thing to have, but validator complains for non-escaped script.

<script type="text/javascript">
function show_adv(box)
{
    var rows = document.getElementsByTagName('TR');
    for(var i=0; i<rows.length; i++)
	if(rows[i].className && rows[i].className.match(/advanced/))
	    rows[i].style.display = box.checked ? (bw.ie ? 'block' : 'table-row') : 'none';
}
</script>

Moving into a separate .js file would be beneficial.

Change History (1)

comment:1 Changed 5 years ago by alec

  • Resolution set to fixed
  • Status changed from new to closed

Fixed in [5f660cd7].

Note: See TracTickets for help on using tickets.