Skip to content

feat(canvas): merge/split cells from the slash menu on cell selection - #1212

Merged
hannessolo merged 3 commits into
mainfrom
cellmenu
Aug 19, 2026
Merged

feat(canvas): merge/split cells from the slash menu on cell selection#1212
hannessolo merged 3 commits into
mainfrom
cellmenu

Conversation

@hannessolo

Copy link
Copy Markdown
Contributor

@aem-code-sync

aem-code-sync Bot commented Aug 5, 2026

Copy link
Copy Markdown

Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch.
In case there are problems, just click the checkbox below to rerun the respective action.

  • Re-sync branch
Commits

@hannessolo
hannessolo marked this pull request as ready for review August 5, 2026 14:19
Classic-editor parity: pressing "/" with one or more table cells selected
now opens a menu offering Merge cells / Split cell (without inserting a "/"),
instead of merge being reachable only from the floating selection toolbar.

Wired into the existing slash-menu plugin, which already owns the nx-menu
element, positioning, and keyboard nav. A `cellMode` flag routes selection to
the command directly (no "/query" text to strip) and keeps syncSlashUi from
closing the menu until the cell selection ends. Cell selection is detected via
duck-typed `$anchorCell` rather than `instanceof CellSelection`, which is
fragile across bundled module copies.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Comment thread blocks/canvas/editor-utils/command-defs.js Outdated
Replace the hardcoded CELL_SELECTION_COMMAND_IDS list with a 'slash-cell'
showIn tag on the merge/split cell commands, so cellSelectionSlashItems
derives its commands via commandsFor() like every other slash/toolbar
surface.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@hannessolo
hannessolo merged commit a27e9e7 into main Aug 19, 2026
4 of 5 checks passed
@hannessolo
hannessolo deleted the cellmenu branch August 19, 2026 13:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants