-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Fixes Salt Cure visual bug if mon fainted by direct attack #3388
Conversation
All good, it's only missing a test. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tests
Test added. |
Salt Cure is fine, but not sure about the other one. Is it actually bad that there are two options that do the same thing? I assume giving TMs is something that's often done when debugging, so it's easier for users to quickly locate this given it's in 2 places. What do others think about it? |
Yeah, I don't mind reverting the change back to normal. I just did it because it was an open issue and relatively easy to fix. And I do agree that it doesn't hurt having both options. |
This reverts commit 4e80451.
Reverted the removal of the debug TM/HM duplicate case |
Fixes #3386
Before:
After: