fix: remove empty objects from a style

dream-num · Dec 17, 2024 · 15811a0 · 15811a0
1 parent 4738a66
commit 15811a0
Showing 1 changed file with 8 additions and 1 deletion.
diff --git a/packages/sheets/src/basics/cell-style.ts b/packages/sheets/src/basics/cell-style.ts
@@ -14,8 +14,8 @@
  * limitations under the License.
  */
 
-import { normalizeTextRuns, Tools } from '@univerjs/core';
 import type { IBorderData, ICellData, IDocumentData, IKeyValue, IParagraph, IStyleData, ITextRun, ITextStyle, Nullable, Styles } from '@univerjs/core';
+import { normalizeTextRuns, Tools } from '@univerjs/core';
 
 export function handleStyle(styles: Styles, oldVal: ICellData, newVal: ICellData) {
     // use null to clear style
@@ -36,6 +36,13 @@ export function handleStyle(styles: Styles, oldVal: ICellData, newVal: ICellData
     // then remove null
     if (merge) {
         Tools.removeNull(merge);
+
+        // remove empty object
+        Object.entries(merge).forEach(([key, val]) => {
+            if (typeof val === 'object' && val !== null && Object.keys(val).length === 0) {
+                delete merge[key as keyof IStyleData];
+            }
+        });
     }
 
     if (Tools.isEmptyObject(merge)) {