Skip to content

refactor(web): remove dead EditorToolbar.svelte - #973

Merged
xarmian merged 1 commit into
PerpetualSoftware:mainfrom
dchaudhari7177:chore/remove-dead-editor-toolbar
Jul 20, 2026
Merged

refactor(web): remove dead EditorToolbar.svelte#973
xarmian merged 1 commit into
PerpetualSoftware:mainfrom
dchaudhari7177:chore/remove-dead-editor-toolbar

Conversation

@dchaudhari7177

Copy link
Copy Markdown
Contributor

Closes #903

web/src/lib/components/editor/EditorToolbar.svelte is never imported or mounted anywhere — confirmed with a repo-wide search (grep -rn EditorToolbar excluding node_modules: the only hit is the file itself). The editor's live insert surfaces (slash menu in block-types.ts, mobile toolbar in Editor.svelte, bubble menu) are untouched.

Pure deletion, no behavior change.

Never imported or mounted anywhere (repo-wide search confirms zero
references). The live insert surfaces are the slash menu, the mobile
toolbar in Editor.svelte, and the selection/bubble menu.

Closes PerpetualSoftware#903
@dchaudhari7177
dchaudhari7177 requested a review from xarmian as a code owner July 19, 2026 04:32
@xarmian

xarmian commented Jul 20, 2026

Copy link
Copy Markdown
Collaborator

Thanks for this, @dchaudhari7177! 🙏 Verified the deletion is clean — repo-wide search confirms zero references to EditorToolbar.svelte, and the captureHtmlBlockSnapshot / flipHtmlBlockToSource helpers it used are still exercised by Editor.svelte and htmlBlock.ts, so nothing else goes dead. Appreciate the tidy, well-scoped cleanup. Merging now.

@xarmian
xarmian merged commit 6fcc20f into PerpetualSoftware:main Jul 20, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants