[html5] r4663 - [e] (0) Comment out the onchange='' handler since it's poor practice. Fixing htt [...]

[whatwg at whatwg.org]

Author: ianh
Date: 2010-02-05 15:00:14 -0800 (Fri, 05 Feb 2010)
New Revision: 4663

Modified:
   complete.html
   index
   source
Log:
[e] (0) Comment out the onchange='' handler since it's poor practice.
Fixing http://www.w3.org/Bugs/Public/show_bug.cgi?id=8679

Modified: complete.html
===================================================================
--- complete.html	2010-02-05 21:06:31 UTC (rev 4662)
+++ complete.html	2010-02-05 23:00:14 UTC (rev 4663)
@@ -43490,9 +43490,9 @@
  <menu type="toolbar">
   <label for="goto">Go to...</label>
   <menu label="Go">
-   <select id="goto"
+   <select id="goto"<!--
            onchange="if (this.options[this.selectedIndex].value)
-                     window.location = this.options[this.selectedIndex].value">
+                     window.location = this.options[this.selectedIndex].value"-->>
     <option value="" selected="selected"> Select site: </option>
     <option value="http://www.apple.com/"> Apple </option>
     <option value="http://www.mozilla.org/"> Mozilla </option>

Modified: index
===================================================================
--- index	2010-02-05 21:06:31 UTC (rev 4662)
+++ index	2010-02-05 23:00:14 UTC (rev 4663)
@@ -43389,9 +43389,9 @@
  <menu type="toolbar">
   <label for="goto">Go to...</label>
   <menu label="Go">
-   <select id="goto"
+   <select id="goto"<!--
            onchange="if (this.options[this.selectedIndex].value)
-                     window.location = this.options[this.selectedIndex].value">
+                     window.location = this.options[this.selectedIndex].value"-->>
     <option value="" selected="selected"> Select site: </option>
     <option value="http://www.apple.com/"> Apple </option>
     <option value="http://www.mozilla.org/"> Mozilla </option>

Modified: source
===================================================================
--- source	2010-02-05 21:06:31 UTC (rev 4662)
+++ source	2010-02-05 23:00:14 UTC (rev 4663)
@@ -48385,9 +48385,9 @@
  <menu type="toolbar">
   <label for="goto">Go to...</label>
   <menu label="Go">
-   <select id="goto"
+   <select id="goto"<!--
            onchange="if (this.options[this.selectedIndex].value)
-                     window.location = this.options[this.selectedIndex].value">
+                     window.location = this.options[this.selectedIndex].value"-->>
     <option value="" selected="selected"> Select site: </option>
     <option value="http://www.apple.com/"> Apple </option>
     <option value="http://www.mozilla.org/"> Mozilla </option>