Skip to content

feat(ew): add support for blocks search in slash menu#1084

Open
usman-khalid wants to merge 2 commits into
mainfrom
ewsta
Open

feat(ew): add support for blocks search in slash menu#1084
usman-khalid wants to merge 2 commits into
mainfrom
ewsta

Conversation

@usman-khalid

@usman-khalid usman-khalid commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Adds support to slash menu to pick blocks from the library if configured.

2026-07-08 12 34 10

Test URL: https://ewsta--da-live--adobe.aem.page/canvas?da-admin=stage&da-collab=stage#/usman-khalid/stage-site-2/comments

Should be merged with: adobe/da-nx#585

@aem-code-sync

aem-code-sync Bot commented Jul 8, 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 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Amazing feature!

});

addSyncedListener(wsProvider, canWrite, setEditable);
prefetchBlocksAfterSync(wsProvider, canWrite);

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wonder if we could wait until the user types / to fetch this? Probably there would be enough time int he time it takes to type the block name?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I thought about this - but IMO power users are pretty quick if they know what they want from the slash menu. Currently prefetches after the doc is loaded which should be fine? @mhaack thoughts?

Comment thread blocks/canvas/editor-utils/block-slash.js
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.

2 participants