fix image link deletion

microsoft · Sep 18, 2023 · e214063 · e214063
1 parent 8f99244
commit e214063
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 4 deletions.
diff --git a/...ges-content-model/roosterjs-content-model-editor/lib/modelApi/edit/utils/deleteSegment.ts b/...ges-content-model/roosterjs-content-model-editor/lib/modelApi/edit/utils/deleteSegment.ts
@@ -26,11 +26,16 @@ export function deleteSegment(
 
     switch (segmentToDelete.segmentType) {
         case 'Br':
-        case 'Image':
         case 'SelectionMarker':
             segments.splice(index, 1);
             return true;
-
+        case 'Image':
+            if (segmentToDelete.link) {
+                segments.splice(index, 2);
+            } else {
+                segments.splice(index, 1);
+            }
+            return true;
         case 'Entity':
             const operation = segmentToDelete.isSelected
                 ? EntityOperation.Overwrite

diff --git a/packages/roosterjs-editor-plugins/lib/plugins/ImageEdit/ImageEdit.ts b/packages/roosterjs-editor-plugins/lib/plugins/ImageEdit/ImageEdit.ts
@@ -479,7 +479,7 @@ export default class ImageEdit implements EditorPlugin {
                 });
 
                 this.shadowSpan.style.verticalAlign = 'bottom';
-                this.shadowSpan.style.fontSize = '24px';
+                wrapper.style.fontSize = '24px';
 
                 shadowRoot.appendChild(wrapper);
             }

diff --git a/packages/roosterjs-editor-plugins/test/imageEdit/imageEditTest.ts b/packages/roosterjs-editor-plugins/test/imageEdit/imageEditTest.ts
@@ -226,7 +226,7 @@ describe('ImageEdit | rotate and flip', () => {
         editor.select(image);
         plugin.setEditingImage(image, ImageEditOperation.Resize);
         expect(editor.getContent()).toBe(
-            '<span style="vertical-align: bottom; font-size: 24px;"><img id="IMAGE_ID_EDITING" src="test"></span>'
+            '<span style="vertical-align: bottom;"><img id="IMAGE_ID_EDITING" src="test"></span>'
         );
     });
 });