Changeset 2790 for trunk/packages/invirt-web

Timestamp:

Dec 27, 2009, 12:21:03 PM (15 years ago)

Author:

broder

Message:

Don't over-escape Javascript in the web templates.

$ doesn't need to be escaped in Mako templates when it's followed by
an open parenthesis.

File:

• trunk/packages/invirt-web/code/templates/list.mako (modified) (3 diffs)

trunk/packages/invirt-web/code/templates/list.mako

@@ -55 +55 @@
           <td>Autoinstall${self.fn.helppopup('Autoinstalls')}</td>
           <td><input type="radio" name="cd_or_auto" id="cd_or_auto_auto"
-                 onchange="\$('cdromlist').value = ''; \$('vmtype-linux').checked = true">
+                 onchange="$('cdromlist').value = ''; $('vmtype-linux').checked = true">
 ${self.fn.autoList(defaults.cdrom, "$('cd_or_auto_auto').checked = true;$('cdromlist').value = '';$('vmtype-linux').checked = true")}
               (experimental; 2-3 minutes, and you have a machine with empty root password.)
@@ -63 +63 @@
           <td>Boot CD</td>
           <td><input type="radio" name="cd_or_auto" id="cd_or_auto_cd" checked="checked"
-                onchange="\$('autoinstalllist').value = ''; \$('vmtype-linux-hvm').checked = true">
+                onchange="$('autoinstalllist').value = ''; $('vmtype-linux-hvm').checked = true">
 ${self.fn.cdromList(defaults.cdrom, "$('cd_or_auto_cd').checked = true;$('autoinstalllist').value = '';$('vmtype-linux-hvm').checked = true")}
 </td>
@@ -148 +148 @@
     <script type="text/javascript">
         document.observe("dom:loaded", function() {
-            stripe(\$('machinelist').getElementsByTagName('table')[0],
+            stripe($('machinelist').getElementsByTagName('table')[0],
                    'stripedrow');
         });