Skip to content

fix(desktop): expose About and Check for Updates in Help menu on Windows/Linux#888

Merged
lefarcen merged 2 commits intomainfrom
fix/help-menu-windows-linux
Apr 8, 2026
Merged

fix(desktop): expose About and Check for Updates in Help menu on Windows/Linux#888
lefarcen merged 2 commits intomainfrom
fix/help-menu-windows-linux

Conversation

@lefarcen
Copy link
Copy Markdown
Collaborator

@lefarcen lefarcen commented Apr 7, 2026

What

Add About Nexu and Check for Updates… entries to the Help menu on Windows and Linux.

Why

On macOS these items live in the application menu by platform convention, so Windows/Linux users currently have no in-app way to see the version or trigger an update check. Refs #784.

How

  • Import dialog from electron and add a small showAboutDialog() helper that surfaces app/Electron/Chromium/Node versions.
  • Refactor installApplicationMenu() to build the Help submenu as an array, then conditionally append Check for Updates… and About Nexu (vX.Y.Z) when process.platform !== \"darwin\".
  • macOS behavior is unchanged — the application menu still owns these items there.

Affected areas

  • Desktop app (Electron shell)

Checklist

  • `pnpm typecheck` passes
  • `pnpm lint` passes
  • `pnpm test` passes
  • No credentials or tokens in code or logs
  • No `any` types introduced

Notes for reviewers

Verified the diff applies cleanly on top of latest `main`. I have not been able to run `pnpm typecheck` / `pnpm lint` locally on this branch — please re-run in CI before merging.

…ows/Linux

On macOS these items live in the application menu by platform convention,
but Windows and Linux have no app menu, leaving users with no way to
view the version or trigger an update check. Add them to the Help menu
on non-darwin platforms.

Refs #784
@sentry
Copy link
Copy Markdown

sentry Bot commented Apr 7, 2026

Codecov Report

❌ Patch coverage is 0% with 49 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
apps/desktop/main/index.ts 0.00% 49 Missing ⚠️

📢 Thoughts on this report? Let us know!

@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented Apr 8, 2026

Deploying nexu-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: b7e5645
Status: ✅  Deploy successful!
Preview URL: https://ac9b0b79.nexu-docs.pages.dev
Branch Preview URL: https://fix-help-menu-windows-linux.nexu-docs.pages.dev

View logs

@lefarcen lefarcen merged commit 5d5e465 into main Apr 8, 2026
18 checks passed
@lefarcen lefarcen mentioned this pull request Apr 8, 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