combined two iterations into one

added null defaults
removed unused var
removed unnecessary destructuring to object

src/reducers/helpers/combine-shapes.js

@@ -47,12 +47,16 @@ const combineShapes = ({
     ),
   };
 
-  return shapes.splice({
-    start: shapes.indexOf(shape2),
-    deleteCount: 1,
-  }).map(
-    (shape) => shape === shape1 ? newShape : shape
-  );
+  return shapes.reduce((newShapes, shape, idx, key) => {
+      switch (true) {
+        case shape === shape2:
+          return newShapes;
+        case shape === shape1:
+          return newShapes.append([key], newShape);
+        default:
+          return newShapes.append([key], shape);
+      }
+    }, List());
 };
 
 combineShapes.JoinTypes = JoinTypes;

src/reducers/root-reducer.js

@@ -154,8 +154,8 @@ export const initialState = {
     enabled: false,
     height: 0,
     width: 0,
-    x: 0,
-    y: 0,
+    x: null,
+    y: null,
   },
   uiOptions: {
     vertexRadius: 6.5,

src/tools/List.js

@@ -279,7 +279,6 @@ const ListPrototype = Object.create({}, {
       const deleteCount = options?.deleteCount ?? 0;
       const newKeys = options?.newKeys ? Array.from(options.newKeys) : [];
       const { keys, values } = this;
-      const uniqueKeys = [...keys, ...newKeys];
 
       for (let i = 0, l = newValues.length; i < l; i++) {
         newKeys[i] = newKeys[i] ?? uniqueKey([...keys, ...newKeys]);

src/tools/combine-reducers.js

@@ -8,7 +8,7 @@
  *
  */
 const combineReducers = (...reducers) => (inState, action) => reducers.reduce(
-  (outState, reducer) => ({ ...reducer(outState, action) }),
+  (outState, reducer) => (reducer(outState, action)),
   inState
 );