Skip to content

Address review feedback for startup account and window creation - #48

Draft
matheuscardosoj wants to merge 1 commit into
karem505:masterfrom
matheuscardosoj:development-pr26-review-fixes
Draft

matheuscardosoj wants to merge 1 commit into
karem505:masterfrom
matheuscardosoj:development-pr26-review-fixes

Conversation

@matheuscardosoj

Copy link
Copy Markdown
Contributor

Scope

Addresses the maintainer feedback that is not already present in master:

  • Persist and restore the preferred startup account, with a minimal settings selector and compatibility for existing account files.
  • Create the settings and lock windows outside synchronous callbacks, serialized to avoid concurrent WebView2 construction.
  • Destroy a lock screen whose creation races a successful unlock.

The current master already includes the asynchronous account-management commands and idempotent autostart disablement, so they are intentionally absent from this focused diff.

This draft replaces #26, which was closed when its head branch was renamed.

Validation

  • cargo test --manifest-path src-tauri/Cargo.toml --locked — 102 passed.
  • cargo clippy --manifest-path src-tauri/Cargo.toml --all-targets -- -D warnings -A unused_mut passed. unused_mut is a pre-existing Windows-only warning in master.
  • git diff --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.

1 participant