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

Usability - Make moving cards easier/less restricted #125

Open
codemonkey-uk opened this issue Feb 8, 2023 · 2 comments
Open

Usability - Make moving cards easier/less restricted #125

codemonkey-uk opened this issue Feb 8, 2023 · 2 comments
Labels
enhancement New feature or improvements planned or in progress.

Comments

@codemonkey-uk
Copy link
Owner

It would be good to provide a way for players who make a mistake in card assignment with an method or opportunity to undo that mistake and move the card to where they really wanted it to go.

@codemonkey-uk codemonkey-uk added the enhancement New feature or improvements planned or in progress. label Feb 8, 2023
@JiFish
Copy link
Collaborator

JiFish commented Feb 8, 2023

That's a tough one. Could any player do it? Only the current active player? or the previous player?

Actions are currently instantly sent to the server, we could delay that and client-side add "Do it" and "undo" buttons. It's an extra click every time, so I think people will just start doing it instinctively and it will lose it's benefit.

We could save the last few game states to the db, and provide a "revert to X" system. Fairly easy server side? Not sure what the ui would be like.

If we wanted a straight "undo" button, server side would have to remember the inputs in to the last action(s), and then we'd need "reverse" versions of each action function. Annoying IMO.

@codemonkey-uk
Copy link
Owner Author

I see two paths to this:

  • full undo, save state in a db after every move and restore it, perhaps using browser back button as the trigger
  • we already support moving object cards between players as a free action, unlock this for all detail types

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or improvements planned or in progress.
Projects
None yet
Development

No branches or pull requests

2 participants