UX improvements: send progress, tables, toast policy, seed-phrase safety#52
Merged
Merged
Conversation
…urs/days The progress panel is a bordered card with a 16px section-scale title and a final "Awaiting confirmation" step that hands off to the pending section below. TimeClock shows the two most significant units (s/m/h/d) and clamps negative elapsed values. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Balance-change and UTXO amount columns (headers included) align flush right so decimal points stack down each column; centered numeric values float by their own width and leave a ragged edge. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Failures of send/update-install/resync/randomness-import stay on screen until dismissed — the user has likely tabbed away during a long operation, and the outcome must not expire unseen. Wrong-password attempts on the lock screen replace their toast via a fixed id instead of stacking, with "Incorrect password" copy. Settings success toasts read "Remote node URL updated." and "Cache file deleted". Copy buttons stop click propagation so copying inside a clickable row never triggers the row action. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
"Seed phrase" is the single term app-wide. Both creation flows carry the full two-part warning (only recovery path + possession equals control), and the add-account modal gates Create behind an "I have written down my seed phrase" attestation that resets if the phrase is regenerated. All three reveal surfaces share one 60s auto-hide (use-seed-hide-timer): revealing restarts the full window instead of stacking timers, manual hide retires it, unmount cleans up. The account menu's seed action is "View seed phrase" (key icon, matching modal title) with a confidentiality/anti-phishing warning, an autofocused password field, and Enter-to-submit. Import flows say "account" consistently, and the confirm-step error tells the user what to do: "Some words are in the wrong place." Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
codewordneptune
changed the title
codewordneptune
changed the title
Sword-Smith
approved these changes
Jul 13, 2026
Comment on lines
+71
to
+73
| {/* Task language over mechanism language: nothing is exported (no | ||
| file/keystore) — the flow reveals the seed phrase for backup or | ||
| verification. */} |
Member
There was a problem hiding this comment.
This PR is a bit to liberal with comments for my taste. This code doesn't need a comment. The text in the <Text> tag already explains what this button does. Otherwise, very nice!
Collaborator
Author
There was a problem hiding this comment.
You are absolutely right, fixed!
Sword-Smith
approved these changes
Jul 15, 2026
… seed grid spacing The Add-account modal's middle content (name, warning, seed grid, reveal controls) scrolls only when the window is too short to show it all; the attestation checkbox and Create button stay pinned below, always reachable. On normal window heights the scroll never engages and the layout is unchanged. The modal uses a small yOffset so it can spend nearly the full window height before the inner scroll kicks in. Seed grids across the app now share the same 8px row gutter: onboarding step 2, both grids of the confirm step (answer slots and word bank), and the Add-account modal. The default 16px gutter spent 80px of pure whitespace across six rows and had drifted between copies. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
codewordneptune
added a commit
that referenced
this pull request
Jul 15, 2026
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Additional collection of UX improvements: a clearer send progress panel, aligned money columns, failure toasts that wait to be seen, and safer seed phrase handling.
Stacked on #51 (base:
cwn/spendable-check), so merge that first; this diff then contains only its own commits.Changes
TimeClockrolls elapsed times up to the two most significant units (s/m/h/d).Verification
tsc --noEmitclean, no new warnings.tauri devhot reload during two days of hands-on testing.🤖 Generated with Claude Code