feat: add dimension type (#117)

apache · Nov 26, 2021 · ac241aa · ac241aa
1 parent a98cde6
commit ac241aa
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 5 deletions.
diff --git a/...emporary_superset_ui/superset-ui/packages/superset-ui-dimension/src/computeMaxFontSize.ts b/...emporary_superset_ui/superset-ui/packages/superset-ui-dimension/src/computeMaxFontSize.ts
@@ -1,9 +1,10 @@
 import getTextDimension, { GetTextDimensionInput } from './getTextDimension';
+import { Dimension } from './types';
 
 function decreaseSizeUntil(
   startSize: number,
-  computeDimension: (size: number) => { height: number; width: number },
-  condition: (dimension: { height: number; width: number }) => boolean,
+  computeDimension: (size: number) => Dimension,
+  condition: (dimension: Dimension) => boolean,
 ): number {
   let size = startSize;
   let dimension = computeDimension(size);

diff --git a/.../temporary_superset_ui/superset-ui/packages/superset-ui-dimension/src/getTextDimension.ts b/.../temporary_superset_ui/superset-ui/packages/superset-ui-dimension/src/getTextDimension.ts
@@ -1,4 +1,4 @@
-import { TextStyle } from './types';
+import { TextStyle, Dimension } from './types';
 
 const SVG_NS = 'http://www.w3.org/2000/svg';
 const STYLE_FIELDS: (keyof TextStyle)[] = [
@@ -21,8 +21,8 @@ const DEFAULT_DIMENSION = { height: 20, width: 100 };
 
 export default function getTextDimension(
   input: GetTextDimensionInput,
-  defaultDimension = DEFAULT_DIMENSION,
-) {
+  defaultDimension: Dimension = DEFAULT_DIMENSION,
+): Dimension {
   const { text, className, style = {}, container = document.body } = input;
 
   const textNode = document.createElementNS(SVG_NS, 'text');

diff --git a/...et-frontend/temporary_superset_ui/superset-ui/packages/superset-ui-dimension/src/types.ts b/...et-frontend/temporary_superset_ui/superset-ui/packages/superset-ui-dimension/src/types.ts
@@ -15,3 +15,8 @@ export interface Margin {
   bottom: number;
   right: number;
 }
+
+export interface Dimension {
+  width: number;
+  height: number;
+}