Skip to content

fix: keep the format toolbar fixed while the note scrolls - #60

Merged
RichardHightower merged 2 commits into
mainfrom
feat/sticky-editor-toolbar
Sep 9, 2026
Merged

fix: keep the format toolbar fixed while the note scrolls#60
RichardHightower merged 2 commits into
mainfrom
feat/sticky-editor-toolbar

Conversation

@RichardHightower

Copy link
Copy Markdown
Contributor

Summary

01M23KYWDKJPH89J1M3TSRM0DB

A long note used to grow the window, so the header and the Bold/Italic toolbar scrolled away with the document.

  • Shell is locked to the viewport (html/body/#root/.app at 100%, overflow hidden)
  • .app-main no longer scrolls as a unit
  • Only .editor-zoom scrolls; toolbar / find / Ask AI stay above it
  • E2E: tall note scrolls, toolbar and header Y unchanged, window.scrollY stays 0

Test plan

  • bun run typecheck / guard:client / bun test src (276)
  • Playwright 73 passed, including the new layout spec
  • CI green

01M23KYWDKJPH89J1M3TSRM0DB

Lock the shell to the viewport so a long note cannot grow the page.
Header and format toolbar stay put; only the editor surface scrolls.
@RichardHightower
RichardHightower merged commit f2a01ae into main Sep 9, 2026
3 of 9 checks passed
@RichardHightower
RichardHightower deleted the feat/sticky-editor-toolbar branch September 9, 2026 17:48
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.

1 participant