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

Add UI to support initial health #69

Open
AlmostMatt opened this issue Apr 12, 2017 · 3 comments
Open

Add UI to support initial health #69

AlmostMatt opened this issue Apr 12, 2017 · 3 comments

Comments

@AlmostMatt
Copy link
Collaborator

Blocked by #61

@ajhyndman ajhyndman added this to Someday/Maybe in proving-grounds Apr 12, 2017
@ajhyndman
Copy link
Owner

ajhyndman commented Apr 12, 2017

I had one further idea for how to approach this. We could let the UI show current HP, without trying to support editing. Then, instead, we could allow you, after teeing up an attack, let you "execute" that turn. Then your heroes hp and special cooldowns would be persisted, and you now have the option to preview what will happen on the next attack.

I kind of like this because it's intuitive, not fiddly, and is how state is changed in the real game. i.e. In the game you always start with full HP on each hero, and you only lose stats via an attack.

I think if you ever want to ask the question: "What happens if my Takumi is attacked while he is at 20hp?", you are really trying to answer the question: "What happens if my Takumi is attacked by Robin, and then attacked again by Cordelia?" I think this system could eliminate the manual transcription and state management.

@ajhyndman ajhyndman added this to the 1.3.0 milestone Apr 15, 2017
@AlmostMatt
Copy link
Collaborator Author

My main complaint with that is that it feels a bit weird to 'apply' combats and 'reset' healths, and it also feels like extra work when a user just wants to test something like miracle, retribution, or vantage.

Also it doesn't work for the Ardent Sacrifice + Desperation combo.

@ajhyndman
Copy link
Owner

That's true, and that's a pretty neat combo, too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
proving-grounds
Someday/Maybe
Development

No branches or pull requests

2 participants