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

Automatically flip next character AMD if previous was rolling. #642

Closed
Kafei56 opened this issue Dec 28, 2024 · 4 comments
Closed

Automatically flip next character AMD if previous was rolling. #642

Kafei56 opened this issue Dec 28, 2024 · 4 comments
Labels
enhancement New feature or request

Comments

@Kafei56
Copy link

Kafei56 commented Dec 28, 2024

Is your feature request related to a problem? Please describe.

No

Describe the solution you'd like

This feature is already available when clicking the muddle or strengthen icon by the AMD (attack modifier deck). It would be nice if it was automatic when drawing a single card as well.

Describe alternatives you've considered

None besides manually flipping the next one.

Additional context

No response

@Kafei56 Kafei56 added the enhancement New feature or request label Dec 28, 2024
Lurkars added a commit that referenced this issue Jan 5, 2025
…latest NodeJS LTS 22, adjust Docker to use clean install
@Lurkars
Copy link
Owner

Lurkars commented Jan 5, 2025

Thanks, added in v0.105.7. Happy if you can verify and close issue if solved for you.

@Lurkars Lurkars assigned Kafei56 and unassigned Lurkars Jan 5, 2025
@Lurkars Lurkars added the to test Should be fixed, but needs proper testing label Jan 5, 2025
@Kafei56
Copy link
Author

Kafei56 commented Jan 6, 2025

It's almost there. If your AMD is freshly shuffled and your first draw is a rolling modifier the following behavior happens: short pause with no animation (as if the rolling is being drawn but is not shown visually), and then the next card is drawn as normal. If you click the AMD settings you can confirm that the rolling card was drawn but not shown.

Lurkars added a commit that referenced this issue Jan 6, 2025
@Lurkars
Copy link
Owner

Lurkars commented Jan 6, 2025

I see, should be fixed now in v0.105.8

@Kafei56
Copy link
Author

Kafei56 commented Jan 7, 2025

Looks good! Thank you!

@Kafei56 Kafei56 closed this as completed Jan 7, 2025
@Lurkars Lurkars removed the to test Should be fixed, but needs proper testing label Jan 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants