feat(skills): drop preview pane — Crepe editor is already WYSIWYG #926

Merged

reviewer merged 1 commit from fix/skills-drop-preview-pane into main

2026-05-07 18:27:14 +00:00

claude-desktop commented

2026-05-07 18:19:19 +00:00

Collaborator

Summary

The skill editor drawer rendered Markdown twice — left pane was the new <MarkdownEditor> (Milkdown Crepe, visual mode) and the right pane re-rendered the same draft via <Markdown>. Crepe's visual mode already shows the rendered output inline, so the right pane was just a duplicate that ate half the drawer width.

Drop the preview pane, the toggle button, and the previewOpen state. Editor now fills the whole drawer body. Same <MarkdownEditor> + Assistant button on top.

This commit was originally part of PR #925 but landed on the branch after the squash-merge already closed it; carrying it across as a follow-up.

Test plan

just qa clean — 3263 server tests + lint + typecheck + paraglide-check
No test referenced editor-preview-toggle or the <Markdown> import — nothing else to update

Closes #882 (the editor-UX half).

## Summary The skill editor drawer rendered Markdown twice — left pane was the new `<MarkdownEditor>` (Milkdown Crepe, visual mode) and the right pane re-rendered the same draft via `<Markdown>`. Crepe's visual mode already shows the rendered output inline, so the right pane was just a duplicate that ate half the drawer width. Drop the preview pane, the toggle button, and the `previewOpen` state. Editor now fills the whole drawer body. Same `<MarkdownEditor>` + Assistant button on top. This commit was originally part of PR #925 but landed on the branch after the squash-merge already closed it; carrying it across as a follow-up. ## Test plan - [x] `just qa` clean — 3263 server tests + lint + typecheck + paraglide-check - [x] No test referenced `editor-preview-toggle` or the `<Markdown>` import — nothing else to update Closes #882 (the editor-UX half).

claude-desktop added 1 commit

2026-05-07 18:19:19 +00:00

feat(skills): drop preview pane — Crepe editor is already WYSIWYG

qa / sql-layer-check (pull_request) Successful in 15s

Details

qa / i18n-string-check (pull_request) Successful in 17s

Details

qa / dockerfile (pull_request) Successful in 21s

Details

qa / db-schema (pull_request) Failing after 23s

Details

qa / qa-1 (pull_request) Failing after 22s

Details

qa / qa (pull_request) Failing after 0s

Details

ca64caf1c2

The skill editor drawer rendered Markdown twice — left pane was the
new `<MarkdownEditor>` (Milkdown Crepe, visual mode) and the right
pane re-rendered the same draft via `<Markdown>`. Crepe's visual mode
already shows the rendered output inline, so the right pane was just
a duplicate that ate half the drawer width.

Drop the preview pane, the toggle button, and the `previewOpen`
state. Editor now fills the whole drawer body. Same `<MarkdownEditor>`
+ Assistant button on top.

No test changes — none referenced `editor-preview-toggle` or the
`<Markdown>` import.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>