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

Take Control applies next turn #28

Open
khonkhortisan opened this issue Nov 18, 2018 · 2 comments
Open

Take Control applies next turn #28

khonkhortisan opened this issue Nov 18, 2018 · 2 comments
Labels
control Switching/maintaining control sanitycheck Should I really make it work this way?

Comments

@khonkhortisan
Copy link
Owner

Right-clicking on a player who is currently autoexploring and using the Take Control context menu item sets control to the clicked player, disables autoexplore, then immediately returns control to the other player. The next time the clicked player gets a turn, they're actually controllable.

Make this not immediately hand back control when disabling autoexplore through Take Control.

@khonkhortisan
Copy link
Owner Author

Is this even necessary? Should autoexplore be more easily cancelable without wasting another player's turn?
Take Control should be removed for players and replaced with Stop Autoexplore/Autorun/Autoresting/Automatic Movement
because you can just waste turns to get back to that player (unless you're bringing a whole crowd with you and want to control a specific one, willing to wait many turns to get there while someone could die in that time, yeah there's not a need for that.)

@khonkhortisan
Copy link
Owner Author

Take Control should be Allow Control Next Turn, and be an instant-cast.
Unless alchemist golems. I haven't gotten to those yet. Postpone.

@khonkhortisan khonkhortisan added sanitycheck Should I really make it work this way? control Switching/maintaining control labels Nov 19, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
control Switching/maintaining control sanitycheck Should I really make it work this way?
Projects
None yet
Development

No branches or pull requests

1 participant