Merge branch 'u/jisong/cmcontentchanged' of https://github.com/micros…

…oft/roosterjs into u/jisong/cmcontentchanged
microsoft · Sep 18, 2023 · 345e724 · 345e724
2 parents d96d200 + 0e5f504
commit 345e724
Show file tree

Hide file tree

Showing 2 changed files with 112 additions and 2 deletions.
diff --git a/packages-content-model/roosterjs-content-model-editor/lib/modelApi/common/mergeModel.ts b/packages-content-model/roosterjs-content-model-editor/lib/modelApi/common/mergeModel.ts
@@ -83,7 +83,7 @@ export function mergeModel(
 
             switch (block.blockType) {
                 case 'Paragraph':
-                    mergeParagraph(insertPosition, block, i == 0, context);
+                    mergeParagraph(insertPosition, block, i == 0, context, options);
                     break;
 
                 case 'Divider':
@@ -125,14 +125,21 @@ function mergeParagraph(
     markerPosition: InsertPoint,
     newPara: ContentModelParagraph,
     mergeToCurrentParagraph: boolean,
-    context?: FormatWithContentModelContext
+    context?: FormatWithContentModelContext,
+    option?: MergeModelOption
 ) {
     const { paragraph, marker } = markerPosition;
     const newParagraph = mergeToCurrentParagraph
         ? paragraph
         : splitParagraph(markerPosition, newPara.format);
     const segmentIndex = newParagraph.segments.indexOf(marker);
 
+    if (option?.mergeFormat == 'none' && mergeToCurrentParagraph) {
+        newParagraph.segments.forEach(segment => {
+            segment.format = { ...(newParagraph.segmentFormat || {}), ...segment.format };
+        });
+        delete newParagraph.segmentFormat;
+    }
     if (segmentIndex >= 0) {
         for (let i = 0; i < newPara.segments.length; i++) {
             const segment = newPara.segments[i];

diff --git a/packages-content-model/roosterjs-content-model-editor/test/modelApi/common/mergeModelTest.ts b/packages-content-model/roosterjs-content-model-editor/test/modelApi/common/mergeModelTest.ts
@@ -2077,6 +2077,109 @@ describe('mergeModel', () => {
         });
     });
 
+    it('Merge with default format paragraph and paragraph, mergeFormat: none', () => {
+        const MockedFormat = {
+            fontFamily: 'sourceSegmentFormatFontFamily',
+            italic: 'sourceSegmentFormatItalic',
+            underline: 'sourceSegmentFormatUnderline',
+            fontSize: 'sourceSegmentFormatFontSize',
+        } as any;
+        const majorModel = createContentModelDocument(MockedFormat);
+        majorModel.blocks.push({
+            blockType: 'Paragraph',
+            segmentFormat: MockedFormat,
+            segments: [
+                {
+                    segmentType: 'Text',
+                    text: 'test',
+                    format: {
+                        fontFamily: 'sourceFontFamily',
+                    } as any,
+                },
+                createSelectionMarker(),
+            ],
+            format: {},
+        });
+        const sourceModel: ContentModelDocument = {
+            blockGroupType: 'Document',
+            blocks: [
+                {
+                    blockType: 'Paragraph',
+                    segments: [
+                        {
+                            segmentType: 'Text',
+                            text: 'test',
+                            format: {
+                                fontFamily: 'sourceFontFamily',
+                                italic: 'sourceItalic',
+                                underline: 'sourceUnderline',
+                                fontSize: 'sourcefontSize',
+                            } as any,
+                        },
+                    ],
+                    format: {},
+                },
+            ],
+        };
+        const para1 = createParagraph();
+        const marker = createSelectionMarker();
+
+        para1.segments.push(marker);
+        majorModel.blocks.push(para1);
+
+        mergeModel(
+            majorModel,
+            sourceModel,
+            { newEntities: [], deletedEntities: [] },
+            {
+                mergeFormat: 'none',
+            }
+        );
+
+        expect(majorModel).toEqual({
+            blockGroupType: 'Document',
+            blocks: [
+                {
+                    blockType: 'Paragraph',
+                    segments: [
+                        Object({
+                            segmentType: 'Text',
+                            text: 'test',
+                            format: Object({
+                                fontFamily: 'sourceFontFamily',
+                                italic: 'sourceSegmentFormatItalic',
+                                underline: 'sourceSegmentFormatUnderline',
+                                fontSize: 'sourceSegmentFormatFontSize',
+                            }),
+                        }),
+                        Object({
+                            segmentType: 'Text',
+                            text: 'test',
+                            format: Object({
+                                fontFamily: 'sourceFontFamily',
+                                italic: 'sourceItalic',
+                                underline: 'sourceUnderline',
+                                fontSize: 'sourcefontSize',
+                            }),
+                        }),
+                        Object({
+                            segmentType: 'SelectionMarker',
+                            isSelected: true,
+                            format: Object({
+                                fontFamily: 'sourceSegmentFormatFontFamily',
+                                italic: 'sourceSegmentFormatItalic',
+                                underline: 'sourceSegmentFormatUnderline',
+                                fontSize: 'sourceSegmentFormatFontSize',
+                            }),
+                        }),
+                    ],
+                    format: {},
+                },
+            ],
+            format: MockedFormat,
+        });
+    });
+
     it('Merge Table + Paragraph', () => {
         const majorModel = createContentModelDocument();
         const sourceModel: ContentModelDocument = {