Add a condition system #59

New issue

Closed

opened 2023-04-24 19:42:12 +00:00 by LunarTides · 0 comments

LunarTides commented

2023-04-24 19:42:12 +00:00

(Migrated from github.com)

Is your feature request related to a problem? Please describe.
Currently cards that have conditions don't always tell you that the condition is cleared or not. This is in vanilla, which gives the card a gold outline.

Describe the description of what you want to happen.
Add a function to the bp card, like a battlecry, called condition. It returns if the condition is cleared or not. This can then be called in EventManager.updateCards, remove (Condition cleared!) if the card has it, and if the function returns true, add (Condition cleared!) in green to the card.

Describe alternatives you've considered
The alternatives are:

Manually add this to every conditioned card.
Don't tell the user that the condition is cleared.
Add a function to functions.js that will automatically apply text to the card depending if the condition is cleared.

None of these alternatives are ideal.

Additional context
None

**Is your feature request related to a problem? Please describe.** Currently cards that have conditions don't always tell you that the condition is cleared or not. This is in vanilla, which gives the card a gold outline. **Describe the description of what you want to happen.** Add a function to the bp card, like a battlecry, called `condition`. It returns if the condition is cleared or not. This can then be called in `EventManager.updateCards`, remove ` (Condition cleared!)` if the card has it, and if the function returns true, add ` (Condition cleared!)` in green to the card. **Describe alternatives you've considered** The alternatives are: 1. Manually add this to every conditioned card. 2. Don't tell the user that the condition is cleared. 3. Add a function to functions.js that will automatically apply text to the card depending if the condition is cleared. None of these alternatives are ideal. **Additional context** None

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

LunarTides/Hearthstone.js#59

No description provided.

Rows
Columns