Skip to content

test: lock document workspace scrolling#1003

Merged
ShenzheZhu merged 1 commit into
mainfrom
test/999-workspace-scroll-regression
Jul 10, 2026
Merged

test: lock document workspace scrolling#1003
ShenzheZhu merged 1 commit into
mainfrom
test/999-workspace-scroll-regression

Conversation

@ShenzheZhu

Copy link
Copy Markdown
Collaborator

Closes #999

Summary

  • add a browser-local Playwright workspace fixture with a long document and multi-page PDF
  • assert the page itself never scrolls while PDF and editor panes scroll independently at 1440x900 and 1024x768
  • keep navbar, editor toolbar, and PDF controls visible after internal scrolling
  • treat demo document IDs as editor routes so the transition wrapper cannot break full-height workspace layout
  • run the regression automatically in CI

Verification

  • pnpm test:e2e:workspace
  • pnpm test:runnable
  • pnpm test:audit
  • pnpm script:audit
  • pnpm type-check
  • pnpm lint
  • pnpm build:all

@ShenzheZhu ShenzheZhu merged commit 8a1f6cc into main Jul 10, 2026
11 checks passed
@ShenzheZhu ShenzheZhu deleted the test/999-workspace-scroll-regression branch July 10, 2026 17:39
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.

Fix document workspace page-level scrolling

1 participant