Skip to content

Add native editable document columns - #427

Merged
MaggieAppleton merged 2 commits into
mainfrom
maggie/native-document-layouts
Oct 9, 2026
Merged

MaggieAppleton merged 2 commits into
mainfrom
maggie/native-document-layouts

Conversation

@MaggieAppleton

@MaggieAppleton MaggieAppleton commented Oct 9, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • Add native, directly editable two-column sections to the restricted MDX dialect and shared Lexical/Yjs registry.
  • Add /columns, keyboard exit, responsive stacking, and content-preserving Unwrap columns.
  • Compose two-image galleries from existing image nodes and editable caption paragraphs; add URL and alt-text repair for existing images.

Preview

These captures show the real editor on the current main layout, with the same gallery side by side in a wide document and stacked in a narrow document.

Checks

  • 4,230 Bun tests and 7 Chromium columns tests pass on the merged branch, including two-reader sync, reconnect, reload, image repair, and no horizontal overflow.
  • bun run types, bun run ci, bun run fix, and the web build pass.

Current limits

This slice accepts exactly two top-level columns. Caption styling applies to the paragraph immediately after an image paragraph; the caption remains independent prose. Planner operations do not yet address a nested column separately. The concurrent #415 guidance is unchanged.

Two-image gallery side by side in the current Chopin editor

The same gallery and captions stacked in a narrow Chopin editor

@MaggieAppleton
MaggieAppleton marked this pull request as ready for review October 9, 2026 13:17
@MaggieAppleton

MaggieAppleton commented Oct 9, 2026 •

Copy link
Copy Markdown
Collaborator Author

@copilot remove committed plans here and remove any tests that just prove the implementation vs. guarding against future regressions

Copilot AI commented Oct 9, 2026

Copy link
Copy Markdown
Contributor

@copilot remove committed plans here and remove any tests that just prove the implementation vs. guarding against future regressions

Removed the committed implementation plan and the duplicate slash-command unit test; the browser test still covers insertion and caret placement. Commit: 4397c1c.

@MaggieAppleton
MaggieAppleton enabled auto-merge (rebase) October 9, 2026 16:49
@MaggieAppleton
MaggieAppleton force-pushed the maggie/native-document-layouts branch 2 times, most recently from 02972cf to 762d8d0 Compare October 9, 2026 17:59
@MaggieAppleton
MaggieAppleton force-pushed the maggie/native-document-layouts branch from 762d8d0 to 0d3c25a Compare October 9, 2026 19:39
@MaggieAppleton
MaggieAppleton merged commit 74385ff into main Oct 9, 2026
3 checks passed
@MaggieAppleton
MaggieAppleton deleted the maggie/native-document-layouts branch October 9, 2026 19:51
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