Skip to content

fix: add spinner feedback to cancel/refresh buttons and navigate back after cancel#786

Merged
dangershony merged 2 commits intomainfrom
fix/cancel-investment-ui-feedback
Apr 25, 2026
Merged

fix: add spinner feedback to cancel/refresh buttons and navigate back after cancel#786
dangershony merged 2 commits intomainfrom
fix/cancel-investment-ui-feedback

Conversation

@dangershony
Copy link
Copy Markdown
Member

Summary

  • Cancel (Step 1 and Step 2) and Refresh buttons now show a spinning indicator and are disabled while processing
  • After a successful cancel, the investment is removed from the Investments collection and the detail view closes (navigates back to portfolio list)
  • Integration test updated to verify actual UI controls: button enabled/disabled state, spinner visibility, collection removal after cancel

Changes

  • InvestmentDetailView.axaml — Added spinner markup (fa-spinner with rotating animation + status text) to all three action buttons, toggled by IsProcessing
  • InvestmentDetailView.axaml.cs — Minor cleanup ensuring IsProcessing always resets
  • PortfolioViewModel.cs — After successful cancel: removes investment from Investments, sets SelectedInvestment = null, updates HasInvestments
  • InvestmentCancellationTest.cs — Cancel phases now verify UI controls (find buttons by name, check IsEnabled, check spinner visibility, verify collection removal)

… after cancel

- Cancel and refresh buttons show spinning indicator and disable during processing
- After successful cancel, investment is removed from collection and detail view closes
- Integration test verifies UI controls (button enabled state, spinner visibility, collection removal)
After each cancel (before and after approval), the test now:
- Navigates to Find Projects and verifies HasInvested=false, IsOpenAndNotInvested=true
- Opens project detail and asserts InvestButton is visible in the UI
- Asserts HasInvestments reflects the updated collection count
@dangershony dangershony merged commit e9e39b3 into main Apr 25, 2026
3 checks passed
@dangershony dangershony deleted the fix/cancel-investment-ui-feedback branch April 25, 2026 22:30
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