Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Archway: Improve overview of the suits #350

Open
SimonN opened this issue Oct 29, 2023 · 0 comments
Open

Archway: Improve overview of the suits #350

SimonN opened this issue Oct 29, 2023 · 0 comments

Comments

@SimonN
Copy link

SimonN commented Oct 29, 2023

In PySolFC's Archway implementation, the arch piles are dealt at random. The redeal button rotates each arch pile. Thus, regardless of how often you click the redeal button, you'll see a mix of suits among the arch's topmost cards.

This UI makes the following common tasks hard:

  1. Play several same-suited cards from adjacent arch piles.
  2. For a given suit, take inventory of all arch cards of this suit.

About the second point: It's common to examine a suit, scanning all arch piles for their number of cards of that suit (always 0, 1, or 2 identical cards) to plan the play:

  • When there are 0 or 1 identical cards in the arch pile, the remaining 1 or 2 will be in the columns, and we must carefully plan which of them to play first. To solve this, we must have a good view over all other cards of the same suit.
  • When there are 2 identical cards in an arch pile, we want to play them immediately. The only reason to hold one back is when foundations meet and we want to reverse cards between them. Knowing about the 2 identical arch cards reduces mental burden, allowing us to focus on other snarls of the deal.

Possible solution: The redeal button should know a cyclic ordering of the four suits, e.g., clubs-spades-hearts-diamonds-clubs, like the arrangement of foundation piles. The redeal button should know about the current suit; this is new mutable state of the patience, but hidden. Whenever we click the redeal button:

  1. The current suit advances to the next suit of the cyclic ordering,
  2. Each arch pile sorts topmost all its cards of the new current suit.
  3. Maybe paint the number "2" on/beside each arch pile that has two identical cards on its top? Or instead fan the arch cards, at least the topmost 2.

I'm happy to discuss alternative UI ideas.

This issue arose from the fixed #332 (Archway: Cannot take desired suit from arch) which @joeraz fixed. We agreed to file the UI improvement ideas as this separate bug. These UI improvements aren't as urgent as that play-blocking bug #332.

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

No branches or pull requests

1 participant