Merge branch 'master' into vnext

packages/roosterjs-editor-dom/lib/index.ts

@@ -42,6 +42,7 @@ export { getNextLeafSibling, getPreviousLeafSibling } from './utils/getLeafSibli
 export { getFirstLeafNode, getLastLeafNode } from './utils/getLeafNode';
 export { default as getTextContent } from './utils/getTextContent';
 export { default as splitTextNode } from './utils/splitTextNode';
+export { default as normalizeRect } from './utils/normalizeRect';
 export { default as toArray } from './utils/toArray';
 export { default as safeInstanceOf } from './utils/safeInstanceOf';
 export { default as readFile } from './utils/readFile';

packages/roosterjs-editor-dom/lib/selection/getPositionRect.ts

@@ -1,4 +1,5 @@
 import createRange from './createRange';
+import normalizeRect from '../utils/normalizeRect';
 import { NodePosition, NodeType, Rect } from 'roosterjs-editor-types';
 
 /**
@@ -52,17 +53,3 @@ export default function getPositionRect(position: NodePosition): Rect {
 
     return null;
 }
-
-function normalizeRect(clientRect: ClientRect): Rect {
-    // A ClientRect of all 0 is possible. i.e. chrome returns a ClientRect of 0 when the cursor is on an empty p
-    // We validate that and only return a rect when the passed in ClientRect is valid
-    let { left, right, top, bottom } = clientRect || <ClientRect>{};
-    return left + right + top + bottom > 0
-        ? {
-              left: Math.round(left),
-              right: Math.round(right),
-              top: Math.round(top),
-              bottom: Math.round(bottom),
-          }
-        : null;
-}

packages/roosterjs-editor-dom/lib/utils/normalizeRect.ts

@@ -0,0 +1,18 @@
+import { Rect } from 'roosterjs-editor-types';
+
+/**
+ * A ClientRect of all 0 is possible. i.e. chrome returns a ClientRect of 0 when the cursor is on an empty p
+ * We validate that and only return a rect when the passed in ClientRect is valid
+ */
+export default function normalizeRect(clientRect: ClientRect): Rect {
+    let { left, right, top, bottom } =
+        clientRect || <ClientRect>{ left: 0, right: 0, top: 0, bottom: 0 };
+    return left + right + top + bottom > 0
+        ? {
+              left: Math.round(left),
+              right: Math.round(right),
+              top: Math.round(top),
+              bottom: Math.round(bottom),
+          }
+        : null;
+}