Merge pull request #190 from rmosolgo/dont-remove-nodes

Robert Mosolgo · Robert Mosolgo · commit 99a1b38b28db · 2015-03-10T10:19:54.000-07:00
fix(ujs) don't remove nodes after unmounting components
diff --git a/lib/assets/javascripts/react_ujs.js.erb b/lib/assets/javascripts/react_ujs.js.erb
@@ -47,8 +47,6 @@
         var node = nodes[i];
 
         React.unmountComponentAtNode(node);
-        // now remove the `data-react-class` wrapper as well
-        node.parentElement && node.parentElement.removeChild(node);
       }
     }
   };

Original file line number	Diff line number	Diff line change
`@@ -47,8 +47,6 @@`
`47`	`47`	`var node = nodes[i];`
`48`	`48`
`49`	`49`	`React.unmountComponentAtNode(node);`
`50`		- // now remove the `data-react-class` wrapper as well
`51`		`- node.parentElement && node.parentElement.removeChild(node);`
`52`	`50`	`}`
`53`	`51`	`}`
`54`	`52`	`};`