Skip to content

fix(parity): align week-grid macOS capture with web demo seed and scroll - #4670

Closed
tyler-dane wants to merge 1 commit into
mainfrom
cursor/mac-parity-week-grid-capture-2ef3
Closed

tyler-dane wants to merge 1 commit into
mainfrom
cursor/mac-parity-week-grid-capture-2ef3

Conversation

@tyler-dane

Copy link
Copy Markdown
Contributor

Fixes #4657

What and why

Week-grid compare PNGs still diverged on demo banner visibility, scroll origin, missing Lunch with Sam, grid chrome, sidebar tips, and Up Next layout. Most gaps were capture-setup (web banner dismiss, scroll math, crop bands, mac parity chrome) plus mac timed-grid gutter/top rule matching web.

  • Web: dismiss demo banner via status row text (Okay is not a button role); instant scroll using demo-seed.json referenceNow; assert Lunch with Sam before capture.
  • macOS XCUITest: double t after viewport settles; assert Lunch + Exercise; parity presentation discards grid drafts and pins focus-page-jump footer tip like web idle sidebar.
  • Compose: skip web in-viewport calendar header in grid-region crops (WEB_GRID_REGION_TOP).
  • macOS TimeGridView: hour-gutter top offset and timed-grid top rule; past-column surface tint unchanged.
  • UpNextCard: min height and chip padding aligned with web card density.

Verify

Local: bun run verify --strict — web/unit/a11y/parity e2e specs pass; one unrelated flake in shift-hold-event-hints.spec.ts (focus timeout) on this VM. macOS Swift builds CI-only.

Pending: desktop-web-parity-screenshots on this PR for updated week-grid-*-compare.png artifacts.

Open in Web Open in Cursor 

@tyler-dane tyler-dane added agent-automerge Agent-loop finished; merge-guard may squash-merge parity-screenshots Run the web vs macOS parity screenshot workflow on this PR labels Oct 21, 2026 — with Cursor
@tyler-dane
tyler-dane enabled auto-merge October 21, 2026 11:07
Co-authored-by: Tyler Dane <tyler-dane@users.noreply.github.com>
@cursor
cursor Bot force-pushed the cursor/mac-parity-week-grid-capture-2ef3 branch from 0cae389 to 1c6521b Compare October 21, 2026 11:08
@tyler-dane
tyler-dane added this pull request to the merge queue Oct 21, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to a conflict with the base branch Oct 21, 2026
@tyler-dane

Copy link
Copy Markdown
Contributor Author

agent-loop: still conflicting after update-branch onto main. Closing so the issue can be relaunched.

@tyler-dane tyler-dane removed the agent-automerge Agent-loop finished; merge-guard may squash-merge label Oct 21, 2026
@tyler-dane tyler-dane closed this Oct 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

parity-screenshots Run the web vs macOS parity screenshot workflow on this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants