Skip to content

fix(swap): block backdrop clicks from closing swap dialog#201

Open
BitHighlander wants to merge 3 commits into
developfrom
fix/swap-dialog-block-backdrop-close
Open

fix(swap): block backdrop clicks from closing swap dialog#201
BitHighlander wants to merge 3 commits into
developfrom
fix/swap-dialog-block-backdrop-close

Conversation

@BitHighlander
Copy link
Copy Markdown
Collaborator

Summary

  • Removes onClick={handleClose} from the outer backdrop Box in SwapDialog.tsx
  • Affects both render paths: the "chain not supported" early-return dialog and the main swap dialog
  • Only the X button (and existing guarded close buttons) can now dismiss the dialog

Test plan

  • Open swap dialog, click outside it — should not close
  • Click X button — should close normally
  • Verify guarded phases (signing/broadcasting/approving) still block X as expected
  • Verify "chain not supported" dialog also requires X to close

PR #186 added Hive to chains.ts with minFirmware 7.14.0 (wrong).
The fix in ae70c35 (PR #187) was not carried through because develop
already had the Hive entry — no conflict was surfaced during merge.

- chains.ts: bump Hive minFirmware 7.14.0 → 7.16.0 (no released
  firmware supports Hive until 7.16.0)
- CommandPalette: add firmwareVersion prop + filter with isChainSupported
  so ⌘K results also respect firmware gates
- App.tsx: pass deviceState.firmwareVersion to CommandPalette
Trigger rebuildActivityHistory in the background (3s after ready) every
time the state-change handler sees state='ready'. Covers both initial
app startup and device reconnects. Passphrase wallets are skipped for
privacy, matching the existing scanChainHistory guard.
Only the X button can now dismiss the swap dialog — prevents accidental
closure mid-flow when clicking outside the modal.
@BitHighlander BitHighlander requested a review from pastaghost as a code owner May 28, 2026 00:44
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