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

Deferred inline roll in messages add to situational bonus #23

Merged
merged 7 commits into from
Apr 11, 2022

Conversation

kaelad02
Copy link
Owner

Found a use-case in my game where the barbarian (path of the zealot) adds some bonus damage while raging, but only on the first attack that hits. I thought about adding a message to his Rage active effect to remind him, but he already remembers. However, making it a little easier would be nice. What if we could add a message and use the deferred inline roll syntax (e.g. [[/r 1d6]]) to make a button that added that roll formula to the situational bonus field? That's what this is designed to do.

I created an active effect to add a message to damage rolls with the following value:

[[/r 1d6+3]]{Divine Fury} damage on first hit

It shows up in the dialog like this:

image

When I click on the "Divine Fury" button, it adds 1d6+3 to the situational bonus field.

kaelad02 added 4 commits March 9, 2022 18:11
Copied styling from inline-roll. Left out some things that we don't need, so not everything that has inline-roll was copied over. Also added some comments to define the different blocks of styling
@kaelad02
Copy link
Owner Author

Can't merge yet as I found a bug with DAE. Well, I think it's a bug anyways. Have to wait and see what happens with this: https://gitlab.com/tposney/dae/-/issues/290

@kaelad02 kaelad02 merged commit 1935db1 into main Apr 11, 2022
@kaelad02 kaelad02 deleted the deferredRollButtons branch April 11, 2022 01:31
@kaelad02 kaelad02 added the enhancement New feature or request label Apr 11, 2022
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

Successfully merging this pull request may close these issues.

1 participant