Skip to content

fix(desktop): add macOS 27 WebKit scroll workaround - #343

Merged
murongg merged 1 commit into
mainfrom
fix/macos-27-webkit-scroll
Jun 19, 2026
Merged

fix(desktop): add macOS 27 WebKit scroll workaround#343
murongg merged 1 commit into
mainfrom
fix/macos-27-webkit-scroll

Conversation

@murongg

@murongg murongg commented Jun 19, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Add a macOS 27-only WebKit scrolling workaround marker based on the desktop OS version.
  • Keep Markra custom scrollbars visible while the workaround is active.
  • Cover platform detection, root attribute cleanup, and scrollbar styles with focused tests.

Why

  • The crash log points at a macOS 27 WebKit scrolling tree failure inside WKWebView.
  • The workaround is scoped to macOS 27.x so other platforms and macOS releases keep their existing behavior.

Validation

  • pnpm --filter @markra/app exec vitest run src/styles.test.ts -t "visible scrollbars" passed
  • pnpm --filter @markra/app exec vitest run src/lib/platform.test.ts passed
  • pnpm --filter @markra/app exec vitest run src/App.test.tsx -t "WebKit scrolling workaround" passed
  • pnpm --filter @markra/app build passed
  • pnpm --filter @markra/desktop build passed
  • pnpm --filter @markra/web build passed

Risk

  • This is a targeted WKWebView workaround, not a root WebKit fix. It can be removed or narrowed when macOS/WebKit behavior stabilizes.

@vercel

vercel Bot commented Jun 19, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
markra-web Ready Ready Preview, Comment Jun 19, 2026 3:19am

@murongg
murongg marked this pull request as ready for review June 19, 2026 03:22
@murongg
murongg merged commit 07fc409 into main Jun 19, 2026
4 checks passed
@murongg
murongg deleted the fix/macos-27-webkit-scroll branch June 19, 2026 03:23
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