[issue2551358] Remove htmll comments inside of script tags.

New submission from John Rouillard:

A lot of the javascript in Roundup looks like:

  <script>
     <!--
    var text_field = document.frm_help.text_preview;
    original_field=form[field].value;
    text_field.value=original_field;
    //-->
  </script>

The html comment marker and (js) commented out end html comment marker
was a trick for netscape 1.0 to not show script code (since it didn't know to
hide stuff inside a script tag). It should be removed everywhere it occurs.

Since we don't have a javascript testing frame work in place, testing the
javascript will have to be manual.

This is a simple set of edits good for a first timer.

ref: https://stackoverflow.com/questions/808816/are-html-comments-inside-script-tags-a-best-
practice

----------
components: Web interface
keywords: Effort-Low, StarterTicket
messages: 8093
nosy: rouilj
severity: minor
status: new
title: Remove htmll comments inside of script tags.
type: behavior

_________________________________________________
Roundup tracker <[email protected]>
<https://issues.roundup-tracker.org/issue2551358>
_________________________________________________