move INHERITABLE_PROPERTIES to roosterjs-editor-types

microsoft · Jun 3, 2024 · 4f489bf · 4f489bf
1 parent 539ffa9
commit 4f489bf
Show file tree

Hide file tree

Showing 4 changed files with 38 additions and 37 deletions.
diff --git a/packages/roosterjs-content-model-dom/lib/modelToDom/optimizers/removeUnnecessaryStyle.ts b/packages/roosterjs-content-model-dom/lib/modelToDom/optimizers/removeUnnecessaryStyle.ts
@@ -1,4 +1,4 @@
-import { INHERITABLE_PROPERTIES } from 'roosterjs-editor-dom/lib/htmlSanitizer/getInheritableStyles';
+import { INHERITABLE_PROPERTIES } from 'roosterjs-editor-types';
 import { isNodeOfType } from '../../domUtils/isNodeOfType';
 
 /**

diff --git a/packages/roosterjs-editor-dom/lib/htmlSanitizer/getInheritableStyles.ts b/packages/roosterjs-editor-dom/lib/htmlSanitizer/getInheritableStyles.ts
@@ -1,15 +1,6 @@
+import { INHERITABLE_PROPERTIES } from 'roosterjs-editor-types';
 import type { StringMap } from 'roosterjs-editor-types';
 
-// Inheritable CSS properties
-// Ref: https://www.w3.org/TR/CSS21/propidx.html
-export const INHERITABLE_PROPERTIES = (
-    'border-spacing,caption-side,color,' +
-    'cursor,direction,empty-cells,font-family,font-size,font-style,font-variant,font-weight,' +
-    'font,letter-spacing,line-height,list-style-image,list-style-position,list-style-type,' +
-    'list-style,orphans,quotes,text-align,text-indent,text-transform,visibility,white-space,' +
-    'widows,word-spacing'
-).split(',');
-
 /**
  * Get inheritable CSS style values from the given element
  * @param element The element to get style from

diff --git a/packages/roosterjs-editor-types/lib/Constants.ts b/packages/roosterjs-editor-types/lib/Constants.ts
@@ -0,0 +1,9 @@
+// Inheritable CSS properties
+// Ref: https://www.w3.org/TR/CSS21/propidx.html
+export const INHERITABLE_PROPERTIES = (
+    'border-spacing,caption-side,color,' +
+    'cursor,direction,empty-cells,font-family,font-size,font-style,font-variant,font-weight,' +
+    'font,letter-spacing,line-height,list-style-image,list-style-position,list-style-type,' +
+    'list-style,orphans,quotes,text-align,text-indent,text-transform,visibility,white-space,' +
+    'widows,word-spacing'
+).split(',');
diff --git a/packages/roosterjs-editor-types/lib/type/index.ts b/packages/roosterjs-editor-types/lib/type/index.ts
@@ -1,26 +1,27 @@
-export {
-    AttributeCallback,
-    AttributeCallbackMap,
-    CssStyleCallback,
-    CssStyleCallbackMap,
-    ElementCallback,
-    StringMap,
-    ElementCallbackMap,
-    PredefinedCssMap,
-} from './htmlSanitizerCallbackTypes';
-export { DOMEventHandlerFunction, DOMEventHandlerObject, DOMEventHandler } from './domEventHandler';
-export { TrustedHTMLHandler } from './TrustedHTMLHandler';
-export { SizeTransformer } from './SizeTransformer';
-export {
-    ArrayItemType,
-    DefinitionBase,
-    StringDefinition,
-    NumberDefinition,
-    BooleanDefinition,
-    ArrayDefinition,
-    ObjectDefinition,
-    ObjectPropertyDefinition,
-    CustomizeDefinition,
-    Definition,
-} from './Definition';
-export { CoreCreator } from './CoreCreator';
+export {
+    AttributeCallback,
+    AttributeCallbackMap,
+    CssStyleCallback,
+    CssStyleCallbackMap,
+    ElementCallback,
+    StringMap,
+    ElementCallbackMap,
+    PredefinedCssMap,
+} from './htmlSanitizerCallbackTypes';
+export { DOMEventHandlerFunction, DOMEventHandlerObject, DOMEventHandler } from './domEventHandler';
+export { TrustedHTMLHandler } from './TrustedHTMLHandler';
+export { SizeTransformer } from './SizeTransformer';
+export {
+    ArrayItemType,
+    DefinitionBase,
+    StringDefinition,
+    NumberDefinition,
+    BooleanDefinition,
+    ArrayDefinition,
+    ObjectDefinition,
+    ObjectPropertyDefinition,
+    CustomizeDefinition,
+    Definition,
+} from './Definition';
+export { CoreCreator } from './CoreCreator';
+export { INHERITABLE_PROPERTIES } from '../Constants';