Use getAttribute when soft matching ids.

Some javascript frameworks like angular can overwrite the tag's id
getter, and comparison can turn into a false negative. Newly added
elements don't have the framework's code initialised yet, and calling id
on those elements can differ from the id assigned via angular. Since
there is no way to have frameworks initialized on the new elements,
calling the `id` can be inconsistent. Using getAttribute("id") ensures
that we always compare the html id attributes.

Author: dombesz

dist/idiomorph-ext.esm.js

@@ -476,7 +476,10 @@ var Idiomorph = (function () {
           // If oldElt has an `id` with possible state and it doesn't match newElt.id then avoid morphing.
           // We'll still match an anonymous node with an IDed newElt, though, because if it got this far,
           // its not persistent, and new nodes can't have any hidden state.
-          (!oldElt.id || oldElt.id === newElt.id)
+          (
+            (!oldElt.getAttribute("id")) ||
+            (oldElt.getAttribute("id") === newElt.getAttribute("id"))
+          )
         );
       }
 

dist/idiomorph-ext.js

@@ -474,7 +474,10 @@ var Idiomorph = (function () {
           // If oldElt has an `id` with possible state and it doesn't match newElt.id then avoid morphing.
           // We'll still match an anonymous node with an IDed newElt, though, because if it got this far,
           // its not persistent, and new nodes can't have any hidden state.
-          (!oldElt.id || oldElt.id === newElt.id)
+          (
+            (!oldElt.getAttribute("id")) ||
+            (oldElt.getAttribute("id") === newElt.getAttribute("id"))
+          )
         );
       }
 

dist/idiomorph.cjs.js

@@ -474,7 +474,10 @@ var Idiomorph = (function () {
           // If oldElt has an `id` with possible state and it doesn't match newElt.id then avoid morphing.
           // We'll still match an anonymous node with an IDed newElt, though, because if it got this far,
           // its not persistent, and new nodes can't have any hidden state.
-          (!oldElt.id || oldElt.id === newElt.id)
+          (
+            (!oldElt.getAttribute("id")) ||
+            (oldElt.getAttribute("id") === newElt.getAttribute("id"))
+          )
         );
       }
 

dist/idiomorph.esm.js

@@ -474,7 +474,10 @@ var Idiomorph = (function () {
           // If oldElt has an `id` with possible state and it doesn't match newElt.id then avoid morphing.
           // We'll still match an anonymous node with an IDed newElt, though, because if it got this far,
           // its not persistent, and new nodes can't have any hidden state.
-          (!oldElt.id || oldElt.id === newElt.id)
+          (
+            (!oldElt.getAttribute("id")) ||
+            (oldElt.getAttribute("id") === newElt.getAttribute("id"))
+          )
         );
       }
 

dist/idiomorph.js

@@ -474,7 +474,10 @@ var Idiomorph = (function () {
           // If oldElt has an `id` with possible state and it doesn't match newElt.id then avoid morphing.
           // We'll still match an anonymous node with an IDed newElt, though, because if it got this far,
           // its not persistent, and new nodes can't have any hidden state.
-          (!oldElt.id || oldElt.id === newElt.id)
+          (
+            (!oldElt.getAttribute("id")) ||
+            (oldElt.getAttribute("id") === newElt.getAttribute("id"))
+          )
         );
       }
 

src/idiomorph.js

@@ -474,7 +474,10 @@ var Idiomorph = (function () {
           // If oldElt has an `id` with possible state and it doesn't match newElt.id then avoid morphing.
           // We'll still match an anonymous node with an IDed newElt, though, because if it got this far,
           // its not persistent, and new nodes can't have any hidden state.
-          (!oldElt.id || oldElt.id === newElt.id)
+          (
+            (!oldElt.getAttribute("id")) ||
+            (oldElt.getAttribute("id") === newElt.getAttribute("id"))
+          )
         );
       }