fix: html deserializer error

byoungd · Jul 4, 2023 · f5a4018 · f5a4018
1 parent 54e5dd1
commit f5a4018
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 5 deletions.
diff --git a/.changeset/unlucky-cars-float.md b/.changeset/unlucky-cars-float.md
@@ -0,0 +1,6 @@
+---
+'@editablejs/plugin-alignment': patch
+'@editablejs/plugin-leading': patch
+---
+
+fix html deserializer error
diff --git a/packages/plugins/alignment/src/deserializer/html.ts b/packages/plugins/alignment/src/deserializer/html.ts
@@ -1,15 +1,15 @@
-import { isDOMHTMLElement } from '@editablejs/models'
+import { isDOMHTMLElement, Element } from '@editablejs/models'
 import { HTMLDeserializerWithTransform } from '@editablejs/deserializer/html'
 import { Align, AlignKeys, AlignValue } from '../interfaces/align'
 export const withAlignHTMLDeserializerTransform: HTMLDeserializerWithTransform = next => {
   return (node, options = {}) => {
-    const { element } = options
-    if (isDOMHTMLElement(node)) {
+    if (isDOMHTMLElement(node) && options.element) {
       let { textAlign, justifyContent } = node.style
       if (node.nodeName === 'CENTER') {
         textAlign = 'center'
       }
-      const align: Align = element as Align
+
+      const align: Align = options.element as Align
       if (
         textAlign &&
         textAlign.toLowerCase() !== AlignValue.Left &&

diff --git a/packages/plugins/leading/src/deserializer/html.ts b/packages/plugins/leading/src/deserializer/html.ts
@@ -5,7 +5,7 @@ import { Leading } from '../interfaces/leading'
 export const withLeadingHTMLDeserializerTransform: HTMLDeserializerWithTransform = next => {
   return (node, options = {}) => {
     const { element } = options
-    if (isDOMHTMLElement(node)) {
+    if (isDOMHTMLElement(node) && element) {
       const { lineHeight } = node.style
       let leading: Partial<Leading> = element as Partial<Leading>
       if (lineHeight) {