-
Notifications
You must be signed in to change notification settings - Fork 792
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
Consume Resource After Successful Spell/Oil Usage #6310
base: master
Are you sure you want to change the base?
Conversation
I need help with the time demo |
It's broken because your PR is broken. Go cast a Heal scroll and then try to do something after. |
Nothing else changed besides Scroll of Identify. Also casting scroll of identify directly from speed spell does not work, as only right clicking a Scroll gives the targeting cursor, not actually casting the spell.
Any brave souls willing to playtest this? Also I cannot figure out how to correctly handle oils. Any help would be appreciated. The problematic line has |
ConsumeSpell(player, spl); | ||
} |
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.
imo avoid adding empty diff like in this file - changes not really relevant to the PR
This PR will redefine the behavior of spells that use a targeting cursor.
Default behavior:
Fixed Scroll behavior
Oils will behave in the same manner, not being consumed until successfully used.
Fixes #6105
Fixes #6324