Skip to content

fix: remove error overlay and loaded tracking causing PDF open failures#35

Merged
jrandolf merged 1 commit intomainfrom
fix/remove-error-overlay
Mar 12, 2026
Merged

fix: remove error overlay and loaded tracking causing PDF open failures#35
jrandolf merged 1 commit intomainfrom
fix/remove-error-overlay

Conversation

@jrandolf
Copy link
Contributor

Summary

  • Removes the loaded flag and error overlay that was causing PDF open failures in v0.1.8+
  • Errors from the PDF viewer are now logged to the console instead of replacing the viewer with a crash screen
  • The overlay manager initialization error triggered the crash screen even on successful loads

Closes #32

Test plan

  • Open a PDF file and verify it loads correctly
  • Verify no "overlay manager has not yet been initialized" error appears

🤖 Generated with Claude Code

Closes #32

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@jrandolf jrandolf enabled auto-merge (squash) March 12, 2026 10:15
@jrandolf jrandolf merged commit 053a47f into main Mar 12, 2026
1 check passed
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.

[Bug Report] PDF fails to open in v0.1.8

1 participant