Skip to content

fix(desktop): use consistent 1280×720 window size#738

Merged
Siri-Ray merged 1 commit intomainfrom
fix/window-size-consistency
Apr 1, 2026
Merged

fix(desktop): use consistent 1280×720 window size#738
Siri-Ray merged 1 commit intomainfrom
fix/window-size-consistency

Conversation

@lefarcen
Copy link
Copy Markdown
Collaborator

@lefarcen lefarcen commented Apr 1, 2026

What

Use a fixed 1280×720 window size for all launches, removing the post-animation resize to 1400×920.

Why

The setup animation plays at 1280×720, then the window abruptly resizes to 1400×920 after completion — the jump feels jarring and inconsistent.

How

  • index.ts: Always create the window at 1280×720 instead of conditionally using 1400×920 for non-setup launches.
  • ipc.ts: On setup:animation-complete, only restore vibrancy and relax minWidth — no longer call setSize / center.

Affected areas

  • Desktop app (Electron shell)

Checklist

  • pnpm typecheck passes
  • pnpm lint passes
  • pnpm test passes
  • pnpm generate-types run (if API routes/schemas changed)
  • No credentials or tokens in code or logs
  • No any types introduced (use unknown with narrowing)

@lefarcen lefarcen marked this pull request as draft April 1, 2026 08:43
@lefarcen lefarcen force-pushed the fix/window-size-consistency branch from 8a9994c to 5583b4b Compare April 1, 2026 08:45
Remove the post-animation resize from 1280×720 to 1400×920.
The window now stays at 1280×720 throughout, eliminating the
jarring size jump after the setup animation completes.
@lefarcen lefarcen marked this pull request as ready for review April 1, 2026 08:48
@lefarcen lefarcen force-pushed the fix/window-size-consistency branch from 5583b4b to 33457b5 Compare April 1, 2026 08:48
@Siri-Ray Siri-Ray merged commit 543699c into main Apr 1, 2026
11 checks passed
@lefarcen lefarcen mentioned this pull request Apr 1, 2026
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