Skip to content

fix(navigation): let the bundled app load in release builds - #163

Merged
lezli01 merged 2 commits into
masterfrom
fix/blank-window-release
Oct 25, 2026
Merged

lezli01 merged 2 commits into
masterfrom
fix/blank-window-release

Conversation

@lezli01

@lezli01 lezli01 commented Oct 25, 2026

Copy link
Copy Markdown
Owner

Problem

After installing the latest release, markpad opens to a blank white window.

The navigation guard added in #160 (08c0909) only lets the main window show the app origin at path / or /index.html. Tauri loads the bundled frontend at tauri://localhost — no trailing slash — and because tauri: is not a special URL scheme, Url::path() is "", not "/". The guard classified the app's own start page as Blocked.

Dev builds were unaffected: the Vite dev URL is http://localhost:1420, which normalizes to /. The existing test used tauri://localhost/ (with slash), so it passed too.

Fix

  • Accept an empty path on the app origin in classify.
  • Add a test for tauri://localhost (failed before the fix with left: Blocked, right: App).

Testing

  • cargo test --lib — 6 passed
  • cargo clippy --all-targets — clean
  • Not yet verified with an installed release build; please check npm run tauri build output launches before cutting the patch release.

🤖 Generated with Claude Code

lezli01 and others added 2 commits October 25, 2026 11:59
Tauri loads the bundled frontend at `tauri://localhost` with no trailing
slash. `tauri:` is not a special URL scheme, so that URL's path is empty
rather than "/", and the navigation guard blocked the app's own start page,
leaving a blank white window. Dev builds were unaffected because the Vite
dev server URL is http and normalizes to "/".

Accept the empty path on the app origin and cover it with a test.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@lezli01
lezli01 merged commit ebd2650 into master Oct 25, 2026
8 checks passed
@lezli01
lezli01 deleted the fix/blank-window-release branch October 25, 2026 11:13
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