Land UX improvements on main (follow-up to #52) - #57
Merged
Conversation
…ety (#52) * ui: send progress panel as a status card; elapsed times roll up to hours/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> * ui: right-align amount columns with tabular figures in History tables 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> * ui: sticky failure toasts for long-running operations; toast copy fixes 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> * ui: seed-phrase safety and terminology "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> * style: prettier Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> * Removed unnecessary comments * ui: keep Add-account Create reachable at any window height; harmonize 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> * Additional comment cleanup --------- Co-authored-by: codewordneptune <245962099+codewordneptune@users.noreply.github.com> Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
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
1 participant
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.
Lands the content of #52 on main. #52 was stacked on cwn/spendable-check and was merged after #51, but its base was never retargeted to main, so the UX work landed on the side branch instead of main. This PR brings that already reviewed and CI-tested content to main; the diff is purely the #52 changes.
See #52 for the full change description: send progress feedback, table alignment, toast policy, and seed-phrase safety improvements.
After merging, both cwn/spendable-check and cwn/ux-polish-round1 can be deleted.