cms-module-client - npm Package Compare versions

Comparing version 0.3.4 to 0.3.5

lib/lib/util.js

@@ -99,7 +99,16 @@ "use strict";
   if (classList.length > 0) selector += "." + classList.join(".");
-  var siblings = findSiblingsOfSameName(el);
+  var siblings = findSiblings(el);
+  var siblingsOfSameName = findSiblingsOfSameName(localName, siblings);
   var nthChild = siblings.indexOf(el) + 1;
-  if (nthChild > 1) selector += ":nth-child(".concat(nthChild, ")");
+  if (nthChild > 1 && siblingsOfSameName.length > 1) selector += ":nth-child(".concat(nthChild, ")");
   return selector;
 }
+
+function findSiblings(el) {
+  var parent = el.parentNode;
+
+  var children = _toConsumableArray(parent.children);
+
+  return children;
+}
 /**
@@ -111,7 +120,15 @@ * Find all siblings of an element with the same tag name.
 
-function findSiblingsOfSameName(el) {
-  var localName = el.localName;
-  var parent = el.parentNode;
-  var children = parent.children;
-  var siblings = [];
+function findSiblingsOfSameName(name, siblings) {
+  return siblings.filter(function (sibling) {
+    return sibling.localName === name;
+  });
+}
+/**
+ * Find a section by its element path.
+ * @param {string} path - The path to identify the section.
+ * @param {Object[]} sections - An array of sections.
+ */
+
+
+function findSection(element, sections) {
   var _iteratorNormalCompletion = true;
@@ -122,5 +139,5 @@ var _didIteratorError = false;
   try {
-    for (var _iterator = children[Symbol.iterator](), _step; !(_iteratorNormalCompletion = (_step = _iterator.next()).done); _iteratorNormalCompletion = true) {
-      var child = _step.value;
-      if (child.localName === localName) siblings.push(child);
+    for (var _iterator = sections[Symbol.iterator](), _step; !(_iteratorNormalCompletion = (_step = _iterator.next()).done); _iteratorNormalCompletion = true) {
+      var section = _step.value;
+      if (section.element === element) return section;
     }
@@ -141,13 +158,6 @@ } catch (err) {
   }
-
-  return siblings;
 }
-/**
- * Find a section by its element path.
- * @param {string} path - The path to identify the section.
- * @param {Object[]} sections - An array of sections.
- */
 
-
-function findSection(element, sections) {
+function findChangedSections(sections) {
+  var changed = [];
   var _iteratorNormalCompletion2 = true;
@@ -160,3 +170,3 @@ var _didIteratorError2 = false;
       var section = _step2.value;
-      if (section.element === element) return section;
+      if (section.edited_text !== section.saved_text) changed.push(section);
     }
@@ -177,30 +187,3 @@ } catch (err) {
   }
-}
 
-function findChangedSections(sections) {
-  var changed = [];
-  var _iteratorNormalCompletion3 = true;
-  var _didIteratorError3 = false;
-  var _iteratorError3 = undefined;
-
-  try {
-    for (var _iterator3 = sections[Symbol.iterator](), _step3; !(_iteratorNormalCompletion3 = (_step3 = _iterator3.next()).done); _iteratorNormalCompletion3 = true) {
-      var section = _step3.value;
-      if (section.edited_text !== section.saved_text) changed.push(section);
-    }
-  } catch (err) {
-    _didIteratorError3 = true;
-    _iteratorError3 = err;
-  } finally {
-    try {
-      if (!_iteratorNormalCompletion3 && _iterator3["return"] != null) {
-        _iterator3["return"]();
-      }
-    } finally {
-      if (_didIteratorError3) {
-        throw _iteratorError3;
-      }
-    }
-  }
-
   return changed;
@@ -255,9 +238,9 @@ }
 function appendTools(tools, toolbar) {
-  var _iteratorNormalCompletion4 = true;
-  var _didIteratorError4 = false;
-  var _iteratorError4 = undefined;
+  var _iteratorNormalCompletion3 = true;
+  var _didIteratorError3 = false;
+  var _iteratorError3 = undefined;
 
   try {
-    for (var _iterator4 = tools[Symbol.iterator](), _step4; !(_iteratorNormalCompletion4 = (_step4 = _iterator4.next()).done); _iteratorNormalCompletion4 = true) {
-      var tool = _step4.value;
+    for (var _iterator3 = tools[Symbol.iterator](), _step3; !(_iteratorNormalCompletion3 = (_step3 = _iterator3.next()).done); _iteratorNormalCompletion3 = true) {
+      var tool = _step3.value;
       if (!tool) continue;
@@ -267,12 +250,12 @@ toolbar.appendChild(tool);
   } catch (err) {
-    _didIteratorError4 = true;
-    _iteratorError4 = err;
+    _didIteratorError3 = true;
+    _iteratorError3 = err;
   } finally {
     try {
-      if (!_iteratorNormalCompletion4 && _iterator4["return"] != null) {
-        _iterator4["return"]();
+      if (!_iteratorNormalCompletion3 && _iterator3["return"] != null) {
+        _iterator3["return"]();
       }
     } finally {
-      if (_didIteratorError4) {
-        throw _iteratorError4;
+      if (_didIteratorError3) {
+        throw _iteratorError3;
       }
@@ -407,22 +390,22 @@ }
 function getShortcut(shortcuts, combo) {
-  var _iteratorNormalCompletion5 = true;
-  var _didIteratorError5 = false;
-  var _iteratorError5 = undefined;
+  var _iteratorNormalCompletion4 = true;
+  var _didIteratorError4 = false;
+  var _iteratorError4 = undefined;
 
   try {
-    for (var _iterator5 = shortcuts[Symbol.iterator](), _step5; !(_iteratorNormalCompletion5 = (_step5 = _iterator5.next()).done); _iteratorNormalCompletion5 = true) {
-      var shortcut = _step5.value;
+    for (var _iterator4 = shortcuts[Symbol.iterator](), _step4; !(_iteratorNormalCompletion4 = (_step4 = _iterator4.next()).done); _iteratorNormalCompletion4 = true) {
+      var shortcut = _step4.value;
       if (shortcut.combo.join(" ") === combo.join(" ")) return shortcut;
     }
   } catch (err) {
-    _didIteratorError5 = true;
-    _iteratorError5 = err;
+    _didIteratorError4 = true;
+    _iteratorError4 = err;
   } finally {
     try {
-      if (!_iteratorNormalCompletion5 && _iterator5["return"] != null) {
-        _iterator5["return"]();
+      if (!_iteratorNormalCompletion4 && _iterator4["return"] != null) {
+        _iterator4["return"]();
       }
     } finally {
-      if (_didIteratorError5) {
-        throw _iteratorError5;
+      if (_didIteratorError4) {
+        throw _iteratorError4;
       }
@@ -437,9 +420,9 @@ }
   var result = {};
-  var _iteratorNormalCompletion6 = true;
-  var _didIteratorError6 = false;
-  var _iteratorError6 = undefined;
+  var _iteratorNormalCompletion5 = true;
+  var _didIteratorError5 = false;
+  var _iteratorError5 = undefined;
 
   try {
-    for (var _iterator6 = cookies[Symbol.iterator](), _step6; !(_iteratorNormalCompletion6 = (_step6 = _iterator6.next()).done); _iteratorNormalCompletion6 = true) {
-      var cookie = _step6.value;
+    for (var _iterator5 = cookies[Symbol.iterator](), _step5; !(_iteratorNormalCompletion5 = (_step5 = _iterator5.next()).done); _iteratorNormalCompletion5 = true) {
+      var cookie = _step5.value;
       var keyValue = cookie.split("=");
@@ -451,12 +434,12 @@ var key = keyValue[0].trim();
   } catch (err) {
-    _didIteratorError6 = true;
-    _iteratorError6 = err;
+    _didIteratorError5 = true;
+    _iteratorError5 = err;
   } finally {
     try {
-      if (!_iteratorNormalCompletion6 && _iterator6["return"] != null) {
-        _iterator6["return"]();
+      if (!_iteratorNormalCompletion5 && _iterator5["return"] != null) {
+        _iterator5["return"]();
       }
     } finally {
-      if (_didIteratorError6) {
-        throw _iteratorError6;
+      if (_didIteratorError5) {
+        throw _iteratorError5;
       }
@@ -463,0 +446,0 @@ }

package.json

 {
   "name": "cms-module-client",
-  "version": "0.3.4",
+  "version": "0.3.5",
   "description": "An editor for html pages. This is the client side part of the JS CMS module tool. It lets you edit a webpage in a WYSIWYG way. Together with the Server side module, it creates nicely integrated content management system for Node JS.",
@@ -5,0 +5,0 @@ "main": "lib/index.js",

No alert changes

Worsened metrics

Total package byte prevSize

72321

decreased by-0.69%

Lines of code

1837

decreased by-0.97%

No dependency changes