Fix issue with <select> nodes

Whenever options get removed, other options gets selected.

This is problematic in some case, but luckily that does not
happen when the select insertBefore option nodes.

This change reduce a bit overall code size, and it might
improve performance in most common cases/browsers.

It also fixes a nasty issue in lighterhtml:
WebReflection/lighterhtml#50

Author: WebReflection

cjs/utils.js

@@ -2,14 +2,8 @@
 const Map = (require('@ungap/essential-map'));
 
 const append = (get, parent, children, start, end, before) => {
-  if ((end - start) < 2)
-    parent.insertBefore(get(children[start], 1), before);
-  else {
-    const fragment = parent.ownerDocument.createDocumentFragment();
-    while (start < end)
-      fragment.appendChild(get(children[start++], 1));
-    parent.insertBefore(fragment, before);
-  }
+  while (start < end)
+    parent.insertBefore(get(children[start++], 1), before);
 };
 exports.append = append;
 

esm/utils.js

@@ -1,14 +1,8 @@
 import Map from '@ungap/essential-map';
 
 export const append = (get, parent, children, start, end, before) => {
-  if ((end - start) < 2)
-    parent.insertBefore(get(children[start], 1), before);
-  else {
-    const fragment = parent.ownerDocument.createDocumentFragment();
-    while (start < end)
-      fragment.appendChild(get(children[start++], 1));
-    parent.insertBefore(fragment, before);
-  }
+  while (start < end)
+    parent.insertBefore(get(children[start++], 1), before);
 };
 
 export const eqeq = (a, b) => a == b;

index.js

@@ -39,11 +39,8 @@ try {
 var Map$1 = self.Map;
 
 var append = function append(get, parent, children, start, end, before) {
-  if (end - start < 2) parent.insertBefore(get(children[start], 1), before);else {
-    var fragment = parent.ownerDocument.createDocumentFragment();
-    while (start < end) {
-      fragment.appendChild(get(children[start++], 1));
-    }parent.insertBefore(fragment, before);
+  while (start < end) {
+    parent.insertBefore(get(children[start++], 1), before);
   }
 };