search

schultzcole / FVTT-Minimal-Rolling-Enhancements-DND5E

Some minimalist enhancements to the core D&D5e rolling workflow. Attempts to stay as close to core as possible while improving convenience.
GNU General Public License v3.0
2 stars 7 forks source link

Auto roll critical damage in specific circumstances #7

Closed schultzcole closed 3 years ago

schultzcole commented 3 years ago

It would be reasonable to automatically roll the damage as a critical when a natrual 20 is rolled in the case where auto rolls for both checks and damage is enabled.

trioderegion commented 3 years ago

Only concern are the few features/items that can interact with critical hits (turning a crit into a normal hit).

I can only think of adamantine armor for this case, but it would require a non-crit damage roll.

schultzcole commented 3 years ago

Hmm, that is valid. I would rather have no automation than have automation that could be wrong. Then again, some might say that if a critical hit is rolled and automatic damage rolls are on, not rolling the damage as critical would be incorrect.

Both cases would require the damage to be re-rolled, but I feel the adamantine/crit cancel case is far less common. So I'm leaning towards automatically rolling damage as critical as long as both auto attacks and auto damage are enabled.

trioderegion commented 3 years ago

And worst case, you just roll the attack again to generate a non-crit hit. Small workaround for an extreme edge case. 👍 No complaints.

schultzcole commented 3 years ago

Resolved in 0.1.4