Don't ask for card order if they are all the same
Closed this issue · 4 comments
GoogleCodeExporter commented
Enhancement request: Whenever all cards being returned to the top of the deck
are the same, just do it, don't bother asking the player to "reorder" them.
Original issue reported on code.google.com by michal.f...@gmail.com on 14 Aug 2012 at 1:04
GoogleCodeExporter commented
I'm not sure what the best way to handle this is. On the one hand, you're
right -- there's no need to reorder the cards you're returning if they're all
the same. On the other hand, I definitely want to know what those cards are.
I'm just going to leave this here for now.
Original comment by August.D...@gmail.com on 14 Aug 2012 at 10:59
GoogleCodeExporter commented
In all cases I can now remember the player knows what those cards are -- before
being asked to reorder them, they had to select them anyway. For instance, if I
play a stack of Alchemists, I am asked to select the cards to be returned, I
click each of the Alchemists, and then I am asked again to reorder those
Alchemists. That second step is unnecessary and removing it does not remove any
information from the player.
(Also, it may be possible to record the returned cards in the log. I have no
idea whether this is currently being done or not.)
Original comment by michal.f...@gmail.com on 15 Aug 2012 at 9:01
GoogleCodeExporter commented
Gotcha. If we're talking about cards going back during the cleanup step,
especially Alchemist, issue 286 is open for that enhancement.
I was also considering cards like Navigator, Scout, Apothecary, etc. After
thinking about it for a while, I think the current UI works fine for those
cards.
Returned cards are being logged now (issue 173). I think that made it into
2.04, but I can't remember for sure and I'm too lazy to check.
Original comment by August.D...@gmail.com on 15 Aug 2012 at 12:43
- Changed state: Duplicate
- Added labels: Type-Enhancement
- Removed labels: Type-Defect
GoogleCodeExporter commented
This is fixed in the current build.
Original comment by August.D...@gmail.com on 15 Aug 2012 at 2:42
- Changed state: Fixed