Skip to content

Commit 121728f

Browse files
authored
Cherry-pick #2347: Add back ContentModelBeforePasteEvent (#2348)
* Content Model: Add back ContentModelBeforePasteEvent (#2347) * Cherry-pick #2347
1 parent cb3299c commit 121728f

File tree

5 files changed

+50
-15
lines changed

5 files changed

+50
-15
lines changed

packages-content-model/roosterjs-content-model-editor/lib/editor/utils/eventConverter.ts

Lines changed: 19 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
import { convertDomSelectionToRangeEx, convertRangeExToDomSelection } from './selectionConverter';
22
import { createDefaultHtmlSanitizerOptions } from 'roosterjs-editor-dom';
3+
import type { ContentModelBeforePasteEvent } from '../../publicTypes/ContentModelBeforePasteEvent';
34
import {
45
KnownAnnounceStrings as OldKnownAnnounceStrings,
56
PasteType as OldPasteType,
@@ -133,20 +134,23 @@ export function oldEventToNewEvent<TOldEvent extends OldEvent>(
133134

134135
case PluginEventType.BeforePaste:
135136
const refBeforePasteEvent = refEvent?.eventType == 'beforePaste' ? refEvent : undefined;
137+
const cmBeforePasteEvent = input as ContentModelBeforePasteEvent;
136138

137139
return {
138140
eventType: 'beforePaste',
139141
clipboardData: input.clipboardData,
140-
customizedMerge: refBeforePasteEvent?.customizedMerge,
141-
domToModelOption: refBeforePasteEvent?.domToModelOption ?? {
142-
additionalAllowedTags: [],
143-
additionalDisallowedTags: [],
144-
additionalFormatParsers: {},
145-
formatParserOverride: {},
146-
processorOverride: {},
147-
styleSanitizers: {},
148-
attributeSanitizers: {},
149-
},
142+
customizedMerge:
143+
cmBeforePasteEvent.customizedMerge ?? refBeforePasteEvent?.customizedMerge,
144+
domToModelOption: cmBeforePasteEvent.domToModelOption ??
145+
refBeforePasteEvent?.domToModelOption ?? {
146+
additionalAllowedTags: [],
147+
additionalDisallowedTags: [],
148+
additionalFormatParsers: {},
149+
formatParserOverride: {},
150+
processorOverride: {},
151+
styleSanitizers: {},
152+
attributeSanitizers: {},
153+
},
150154
eventDataCache: input.eventDataCache,
151155
fragment: input.fragment,
152156
htmlAfter: input.htmlAfter,
@@ -349,7 +353,7 @@ export function newEventToOldEvent(input: NewEvent, refEvent?: OldEvent): OldEve
349353
const refBeforePasteEvent =
350354
refEvent?.eventType == PluginEventType.BeforePaste ? refEvent : undefined;
351355

352-
return {
356+
const oldBeforePasteEvent: ContentModelBeforePasteEvent = {
353357
eventType: PluginEventType.BeforePaste,
354358
clipboardData: input.clipboardData,
355359
eventDataCache: input.eventDataCache,
@@ -360,8 +364,12 @@ export function newEventToOldEvent(input: NewEvent, refEvent?: OldEvent): OldEve
360364
pasteType: PasteTypeNewToOld[input.pasteType],
361365
sanitizingOption:
362366
refBeforePasteEvent?.sanitizingOption ?? createDefaultHtmlSanitizerOptions(),
367+
domToModelOption: input.domToModelOption,
368+
customizedMerge: input.customizedMerge,
363369
};
364370

371+
return oldBeforePasteEvent;
372+
365373
case 'beforeSetContent':
366374
return {
367375
eventType: PluginEventType.BeforeSetContent,

packages-content-model/roosterjs-content-model-editor/lib/index.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@ export {
1010
export { IContentModelEditor, ContentModelEditorOptions } from './publicTypes/IContentModelEditor';
1111
export { ContextMenuPluginState } from './publicTypes/ContextMenuPluginState';
1212
export { ContentModelCorePluginState } from './publicTypes/ContentModelCorePlugins';
13+
export { ContentModelBeforePasteEvent } from './publicTypes/ContentModelBeforePasteEvent';
1314

1415
export { ContentModelEditor } from './editor/ContentModelEditor';
1516
export { isContentModelEditor } from './editor/isContentModelEditor';
Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,20 @@
1+
import type { BeforePasteEvent } from 'roosterjs-editor-types';
2+
import type {
3+
DomToModelOptionForPaste,
4+
MergePastedContentFunc,
5+
} from 'roosterjs-content-model-types';
6+
7+
/**
8+
* A temporary event type to be compatible with both legacy plugin and content model editor
9+
*/
10+
export interface ContentModelBeforePasteEvent extends BeforePasteEvent {
11+
/**
12+
* domToModel Options to use when creating the content model from the paste fragment
13+
*/
14+
readonly domToModelOption: DomToModelOptionForPaste;
15+
16+
/**
17+
* customizedMerge Customized merge function to use when merging the paste fragment into the editor
18+
*/
19+
customizedMerge?: MergePastedContentFunc;
20+
}

packages-content-model/roosterjs-content-model-editor/test/editor/utils/eventConverterTest.ts

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ import {
88
} from 'roosterjs-editor-types';
99
import type { ContentChangedEvent, PluginEvent as OldEvent } from 'roosterjs-editor-types';
1010
import type { PluginEvent as NewEvent } from 'roosterjs-content-model-types';
11+
import type { ContentModelBeforePasteEvent } from '../../../lib/publicTypes/ContentModelBeforePasteEvent';
1112

1213
describe('oldEventToNewEvent', () => {
1314
function runTest(
@@ -744,7 +745,9 @@ describe('newEventToOldEvent', () => {
744745
preserveHtmlComments: false,
745746
unknownTagReplacement: null,
746747
},
747-
}
748+
customizedMerge: mockedCustomizedMerge,
749+
domToModelOption: mockedDomToModelOption,
750+
} as ContentModelBeforePasteEvent
748751
);
749752
});
750753

@@ -792,7 +795,9 @@ describe('newEventToOldEvent', () => {
792795
htmlAttributes: mockedHTmlAttributes,
793796
pasteType: PasteType.AsImage,
794797
sanitizingOption: mockedSanitizeOption,
795-
}
798+
customizedMerge: mockedCustomizedMerge,
799+
domToModelOption: mockedDomToModelOption,
800+
} as ContentModelBeforePasteEvent
796801
);
797802
});
798803

versions.json

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44
"packages-content-model": "0.24.0",
55
"overrides": {
66
"roosterjs-editor-core": "8.59.1",
7-
"roosterjs-editor-plugins": "8.60.0"
7+
"roosterjs-editor-plugins": "8.60.0",
8+
"roosterjs-content-model-editor": "8.24.1"
89
}
9-
}
10+
}

0 commit comments

Comments
 (0)