Faster bounds clone

react-grid-layout · Feb 10, 2016 · 6163ad2 · 6163ad2
1 parent 9137b6b
commit 6163ad2
Showing 1 changed file with 12 additions and 1 deletion.
diff --git a/lib/utils/positionFns.es6 b/lib/utils/positionFns.es6
@@ -6,7 +6,8 @@ export function getBoundPosition(draggable, clientX, clientY) {
   // If no bounds, short-circuit and move on
   if (!draggable.props.bounds) return [clientX, clientY];
 
-  let bounds = JSON.parse(JSON.stringify(draggable.props.bounds));
+  // Clone new bounds
+  let bounds = cloneBounds(draggable.props.bounds);
   let node = ReactDOM.findDOMNode(draggable);
 
   if (typeof bounds === 'string') {
@@ -63,3 +64,13 @@ export function getControlPosition(e) {
     clientY: position.clientY
   };
 }
+
+// A lot faster than stringify/parse
+function cloneBounds(bounds) {
+  return {
+    left: bounds.left,
+    top: bounds.top,
+    right: bounds.right,
+    bottom: bounds.bottom
+  };
+}