Skip to content

Sell all#4

Merged
shorres merged 6 commits intomainfrom
sellAll
Oct 22, 2025
Merged

Sell all#4
shorres merged 6 commits intomainfrom
sellAll

Conversation

@shorres
Copy link
Copy Markdown
Owner

@shorres shorres commented Oct 22, 2025

Sell All Portfolio Feature

  • Added a prominent SELL ALL button to the portfolio section, styled to be highly visible and always bright red. [1] [2]
  • Implemented three modals for the "Sell All" process: a nuclear warning confirmation, a second "are you sure" confirmation, and a success summary modal displaying gross, fee, net proceeds, and new wallet balance.
  • Extended the UIManager class to manage new modal elements, handle all modal interactions, and wire up the "Sell All" button and confirmation workflow. [1] [2] [3]

Game Logic Enhancements

  • Added sellAllCards and sellAllPortfolioCards methods to the GameEngine class, which process the sale of all cards (respecting lock settings), update wallet and stats, and return detailed results for the UI.

Other Updates

  • Bumped the app version to 1.6.0 to reflect the new feature.
  • Minor formatting adjustment to the collection display container for better readability.

@shorres shorres merged commit c0156c7 into main Oct 22, 2025
@shorres shorres deleted the sellAll branch October 22, 2025 19:33
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