Support table edit with logical root

[JiuqingSong]

Most features requires that the editing target should be in current logical root, but table edit is an exception. We should still allow showing editing related ui (inserted, resizer, ...) even the table is not in current logical root.

So I'm making this change to allow passing in a callback function to help choose which tables are allowed to be edited, alone with its logical root. Then let each table edit feature set logical root when start to use it (mouse down or click).

[Copilot]

Pull Request Overview

This PR extends table editing to respect a configurable “logical root” by introducing a TableWithRoot type, updating the plugin and editor to accept it, and adjusting related callbacks.

• Adds a logicalRoot parameter throughout table editing APIs and tests
• Splits the single onInsert callback into onBeforeInsert/onAfterInsert
• Introduces TableWithRoot & a default selector in the plugin, and updates exports

Reviewed Changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
packages/roosterjs-content-model-plugins/test/tableEdit/tableMoverTest.ts	Updated TableEditor constructor calls to include logicalRoot
packages/roosterjs-content-model-plugins/test/tableEdit/tableInserterTest.ts	Updated createTableInserter calls to match new callback signature
packages/roosterjs-content-model-plugins/test/tableEdit/tableEditorTest.ts	Added null for logicalRoot in TableEditor constructor calls
packages/roosterjs-content-model-plugins/test/tableEdit/tableEditPluginTest.ts	Changed setTableEditor(table) calls to setTableEditor({ table, logicalRoot })
packages/roosterjs-content-model-plugins/lib/tableEdit/editors/features/TableInserter.ts	Renamed onInsert → onBeforeInsert/onAfterInserted
packages/roosterjs-content-model-plugins/lib/tableEdit/editors/TableEditor.ts	Added logicalRoot field and wired up pre/post-insert hooks
packages/roosterjs-content-model-plugins/lib/tableEdit/TableWithRoot.ts	New interface defining { table, logicalRoot }
packages/roosterjs-content-model-plugins/lib/tableEdit/TableEditPlugin.ts	Switched to TableWithRoot, introduced tableSelector & default
packages/roosterjs-content-model-plugins/lib/index.ts	Exported the new TableWithRoot interface

Comments suppressed due to low confidence (2)

packages/roosterjs-content-model-plugins/lib/tableEdit/editors/features/TableInserter.ts:38

• The parameter is named onAfterInserted while the handler property is onAfterInsert; renaming them consistently (e.g. both onAfterInsert) will reduce confusion.

    onAfterInserted: () => void,

packages/roosterjs-content-model-plugins/test/tableEdit/tableEditorTest.ts:170

• There is no test verifying that editor.setLogicalRoot is called with the provided logicalRoot on edit start; consider adding a unit test for the onBeforeInsert hook.

tEditor = new TableEditor(editor, table, null,

[Copilot]

Restricting logicalRoot to HTMLDivElement could prevent usage with other container types; consider using HTMLElement | null for broader compatibility.

Suggested change

	logicalRoot: HTMLDivElement | null;
	logicalRoot: HTMLElement | null;

[Copilot]

Using the spread operator here (...table) may unintentionally include extra properties; prefer an explicit object literal ({ table: table.table, logicalRoot: table.logicalRoot, rect }).

Suggested change

	...table,
	table: table.table,
	logicalRoot: table.logicalRoot,