Allow skipping marking hasNewContent in formatContentModel

[JiuqingSong]

We have a parameter skipUndoSnapshot in formatContentModel context object to allow skipping adding undo snapshot. When it is set to true, formatContentModel will not add undo snapshot but it will mark hasNewContent to be true so next time undo is performed, we will first add an undo snapshot then undo.

But in some scenario we don't even want to mark hasNewContent as well, so in this PR I modified this parameter to let it allow 3 different values in addition to existing true | false value:

• When set to 'DoNotSkip', it will add undo snapshot (default behavior)
• When set to 'MarkNewContent', it will skip adding undo snapshot but mark "hasNewContent" so that next undo snapshot will be added
• When set to 'SkipAll', it will skip adding undo snapshot and not mark "hasNewContent", as if no change is made

[Copilot]

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (1)

packages/roosterjs-content-model-core/lib/coreApi/formatContentModel/formatContentModel.ts:51

• [nitpick] The variable name 'minMaxImageSize' in the 'handleImages' function is ambiguous. It should be renamed to 'minImageSize'.

const shouldAddSnapshot = (!skipUndoSnapshot || skipUndoSnapshot == 'DoNotSkip') && !isNested;

[Copilot]

The comment for the skipUndoSnapshot parameter is unclear. It should be clarified that true is equivalent to 'MarkNewContent'.