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

Handle dividing multiple damage dice between targets #25

Open
mcglincy opened this issue May 21, 2022 · 2 comments
Open

Handle dividing multiple damage dice between targets #25

mcglincy opened this issue May 21, 2022 · 2 comments

Comments

@mcglincy
Copy link
Collaborator

mcglincy commented May 21, 2022

pg 122: IF THE WEAPON HAS SEVERAL DAMAGE DICE, YOU CAN DIVIDE THE DICE AMONG SEVERAL CLOSE TARGETS.

Assuming that this also mean separate to-hit rolls? (Could add an editable field for damage vs. this target to allow down-sizing roll, or something.) Although in the existing implementation, multiple attack rolls would also mean multiple ammo deductions, which isn't incorrect. Hrm.

Could also just leave single attack roll, and let players/GM manually split the damage dice themselves. However, we're not showing the separate roll.terms.results of "2d4". Maybe we can leverage the click-to-expand features of the default roll chat message?

@mcglincy
Copy link
Collaborator Author

see also classes Roll and Dice, expandInlineResult(), and resources/app/templates/dice/*.html

@asherett
Copy link

Pretty sure it's not meant to be multiple attack rolls (keep in mind rolls are risky in DiS), but completely agree with the suggestion. Ability to click the damage field in the chat card and see the actual rolls (like in dnd5e) would be great.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants