Merge pull request #60 from GreenGremlin/bugfix/textarea-not-updating

Fixing textarea value changes not showing

Author: johanneswilm

diffDOM.js

@@ -707,7 +707,9 @@
                         objNode.attributes[attribute.name] = attribute.value;
                     }
                 }
-                if (aNode.childNodes && aNode.childNodes.length > 0) {
+                if (objNode.nodeName === 'TEXTAREA') {
+                    objNode.value = aNode.value;
+                } else if (aNode.childNodes && aNode.childNodes.length > 0) {
                     objNode.childNodes = [];
                     nodeArray = Array.prototype.slice.call(aNode.childNodes);
                     length = nodeArray.length;
@@ -1110,10 +1112,6 @@
                     break;
                 case this._const.modifyTextElement:
                     node.data = diff[this._const.newValue];
-
-                    if (parentNode.nodeName === 'TEXTAREA') {
-                        parentNode.value = diff[this._const.newValue];
-                    }
                     break;
                 case this._const.modifyValue:
                     node.value = diff[this._const.newValue];

tests/form.html

@@ -192,7 +192,29 @@ <h1>Form element test for diffDOM</h1>
 
     theDiff = dd.diff(first, second);
 
-    if (theDiff.length === 2) {
+    if (theDiff.length === 1) {
+      success(theDiff);
+    } else {
+      testFailure();
+      console.log(theDiff);
+    }
+
+    reportDiv();
+
+    second.innerText = 'Some text';
+    third.innerText = 'Some other text';
+
+    reportDivDescription(first, second);
+
+    theDiff = dd.diff(first, second);
+
+    dd.apply(first, theDiff);
+
+    theDiff = dd.diff(first, third);
+
+    dd.apply(first, theDiff);
+
+    if (first.value === third.value) {
       success(theDiff);
     } else {
       testFailure();