Fix a bug in the example for explianing automatic DOM transactions ignores DOM changes made
authorRyosuke Niwa <rniwa@webkit.org>
Tue, 29 May 2012 22:49:47 -0700
changeset 5 9590e5006d15
parent 4 df99f18b2937
child 6 ac6ab5cf2890
Fix a bug in the example for explianing automatic DOM transactions ignores DOM changes made
outside of the scope per Sukolsak Sakshuwong's comment. Also add him to the list of acknowledgements.
source.html
--- a/source.html	Tue May 29 22:44:17 2012 -0700
+++ b/source.html	Tue May 29 22:49:47 2012 -0700
@@ -19,7 +19,7 @@
 
 <dt>Acknowledgements</dt>
 <dd>Anne van Kesteren, Annie Sullivan, Alex Russell, Alex Vincent, Aryeh Gregor, Ehsan Akhgari, Eric Uhrhane,
-Frederico Caldeira Knabben, Ian Hickson, Johan "Spocke" Sörlin, Jonas Sicking, Ojan Vafai, Olli Pettay</dd>
+Frederico Caldeira Knabben, Ian Hickson, Johan "Spocke" Sörlin, Jonas Sicking, Ojan Vafai, Olli Pettay, Sukolsak Sakshuwong</dd>
 
 <dt>Latest version:</dt>
 <dd><a href="http://dvcs.w3.org/hg/undomanager/raw-file/tip/undomanager.html">
@@ -653,7 +653,8 @@
     </div>
 
     <div class="example">
-        <p>All <a href="#dom-changes">DOM changes</a> made outside of the <a href="#undo-scope">undo scope</a> is ignored.
+        <p>All <a href="#dom-changes">DOM changes</a> made outside of the <a href="#undo-scope">undo scope</a> take effect immediately
+        but are ignored for the purpose of undo and redo.
         In the following example, <code><a href="#dom-undomanager-undo">undo</a>()</code> will only remove "bar" and "foo"
         remains in the body.</p>
 
@@ -666,7 +667,7 @@
     scope.appendChild("bar");
 }});
 scope.undoManager.undo();
-alert(document.body.textContent); // Alerts "bar".
+alert(document.body.textContent); // Alerts "foo".
 </pre>
     </div>