Skip to content

fix: dialog body padding via DialogPanel; restore unlock state on renderer reload - #3

Merged
pyramation merged 1 commit into
mainfrom
fix/dialog-panel-padding
Jul 26, 2026
Merged

fix: dialog body padding via DialogPanel; restore unlock state on renderer reload#3
pyramation merged 1 commit into
mainfrom
fix/dialog-panel-padding

Conversation

@pyramation

Copy link
Copy Markdown
Contributor

Summary

Blocks' DialogContent provides no inner padding of its own — body content is expected to be wrapped in DialogPanel (which supplies px-6, scroll area, and header/footer-aware vertical padding). The New-item and otpauth-import dialogs used a plain <div>, so form fields ran edge-to-edge against the popup border.

 <DialogHeader>…</DialogHeader>
-<div className="flex flex-col gap-4">…</div>
+<DialogPanel className="flex flex-col gap-4">…</DialogPanel>
 <DialogFooter>…</DialogFooter>

This also gives the footer its intended border-t bg-muted/50 strip styling.

Also: App now queries dcrypt.vault.status() on mount, so a renderer reload while the main-process vault is unlocked no longer strands the user on the unlock screen.

After:

fixed dialog

Link to Devin session: https://app.devin.ai/sessions/04636534e07048089ffb6b78142e12cd
Requested by: @pyramation

@pyramation pyramation self-assigned this Jul 26, 2026
@devin-ai-integration

Copy link
Copy Markdown

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment, CI, and merge conflict monitoring

@pyramation
pyramation merged commit 8572cd0 into main Jul 26, 2026
4 checks 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