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

Update Card.InHandCount for PlayerDrawn deck. #619

Merged
merged 1 commit into from
Jan 31, 2015
Merged

Update Card.InHandCount for PlayerDrawn deck. #619

merged 1 commit into from
Jan 31, 2015

Conversation

jw013
Copy link
Contributor

@jw013 jw013 commented Jan 31, 2015

A single commit to address #601. Commit message copied below.

Originally, card drawn / play / get methods only updated
card.InHandCount for PlayerDeck, not PlayerDrawn. This commit updates
PlayerDrawn as well to make the "Use No Deck" overlay behavior more
consistent with the overlay behavior for when a pre-configured deck is
displayed. Specifically, cards in hand are now highlighted the same way
whether a pre-configured deck is used or not.

Originally, card drawn / play / get methods only updated
card.InHandCount for PlayerDeck, not PlayerDrawn. This commit updates
PlayerDrawn as well to make the "Use No Deck" overlay behavior more
consistent with the overlay behavior for when a pre-configured deck is
displayed. Specifically, cards in hand are now highlighted the same way
whether a pre-configured deck is used or not.
@azeier
Copy link
Member

azeier commented Jan 31, 2015

Perfect, thank you.

azeier pushed a commit that referenced this pull request Jan 31, 2015
Update Card.InHandCount for PlayerDrawn deck.
@azeier azeier merged commit 85ba766 into HearthSim:master Jan 31, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants