Skip to content

Fix 17 UI issues from testing comments (Tiers 1-3)#773

Merged
dangershony merged 4 commits intomainfrom
fix/ui-comments-tier1-2-3
Apr 20, 2026
Merged

Fix 17 UI issues from testing comments (Tiers 1-3)#773
dangershony merged 4 commits intomainfrom
fix/ui-comments-tier1-2-3

Conversation

@dangershony
Copy link
Copy Markdown
Member

Summary

  • Fixes 17 of 23 UI issues documented in docs/UI_TESTING_COMMENTS.md (Tiers 1-3)
  • Each fix includes integration test assertions to prevent regressions
  • All tests pass against local Docker signet infrastructure

Fixed Issues

Tier 1 (quick wins)

Tier 2 (medium effort, high value)

Tier 3 (medium effort, medium value)

Remaining (Tier 4 — separate PR)

#4, #10, #11, #19, #20, #21, #23

Address UI bugs and UX gaps documented in docs/UI_TESTING_COMMENTS.md:

Tier 1: #18 stage percentage 0%, #9 invest button spinner,
#2 refresh loading state, #12 debug prefill stage dates

Tier 2: #1 create wallet spinner, #3 funders refresh button,
#5 investor detail refresh, #8 stages refresh after investing,
#14 recovery error messages, #16 stage status after recovery,
#22 claimable stage info text

Tier 3: #6 view transaction link, #7 refresh during approval wait,
#13 recovery stage count filter, #15 wallet refresh before recovery,
#17 spend stage popup timing, #24 below-threshold penalty routing

All changes verified by integration tests against local Docker signet.
Add 'Recovered after penalty', 'Spent by investor', and
'Project Unfunded, Spent back to investor' to IsStatusRecovered so
stage badges render correctly after all recovery paths.

Add regression test assertions in MultiFundClaimAndRecoverTest and
MultiInvestClaimAndRecoverTest to verify stages show non-blank status
after penalty release and unfunded release respectively.
Resolve 3 conflicts:
- CreateProjectViewModel.cs: take staggered stage dates with DateTime.UtcNow
- FundersView.axaml.cs: keep refresh spinner + toast forwarding from PR #773
- CreateWalletModal.axaml: keep main's mobile-friendly Panel spinner
@dangershony dangershony merged commit 6393a1b into main Apr 20, 2026
1 check passed
@dangershony dangershony deleted the fix/ui-comments-tier1-2-3 branch April 20, 2026 23:55
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