multiroll damage roll messages

CCalego9/20/2022
Messing with multiroll chat message in v10
CCalego9/20/2022
demoing the 'edge cases' this supports
CCalego9/20/2022
"Maximize Dice" + "Situational Bonus" + "Extra Critical Damage" + Deselecting one of the damage parts for this damage roll
CCalego9/20/2022
slap a total on the bottom and you've got everything you used to have, and more.
UUUnknown User9/20/2022
7 Messages Not Public
Sign In & Join Server To View
CCalego9/21/2022
Under the hood it creates one DamageRoll instance per part provided to a new dice class: DamageGroupRoll.

That internally looks very similar to DamageRoll, but basically just curates and manages this group of individual rolls.

In this POC I was using the roll.render method on each individual roll, then stitching them into one message
CCalego9/21/2022
I did this with a branch in the system code, it could probably be its own module in the interim though, might not interfere with roll groups
CCalego9/21/2022
Which issue is this?

Yeah ideally it'd allow each individual roll to be right clicked, as well as the total
CCalego9/21/2022
For how I actually implemented this, the entry point is a method that has the exact same call signature as damageRoll. Or at least one that's really close. Need to review the diff again to be sure

Instead of combining the provided parts, I keep them separate for longer in the process, and allow the configure roll dialog to filter those parts further, if the user wants.
CCalego9/21/2022
Long story short, roll groups would "just work" because all it does is curate which parts make it into the appropriate method (iirc)