Delves. Changes to stress rolls.

hitcherland / FoundryVTT-Heart

Basic FoundryVTT System for Heart - The City Beneath

Creative Commons Zero v1.0 Universal

7 stars 14 forks source link

Delves. Changes to stress rolls. #50

Closed Wigginns closed 1 year ago

Wigginns commented 1 year ago

Added properties to Delves and updated the Delve sheet so users can modify and display these values.

Addresses https://github.com/hitcherland/FoundryVTT-Heart/issues/45

Selecting stress type while rolling amount is an option enabled by default. Users can also leave the type blank and it will prompt them for one before applying it.

Wigginns commented 1 year ago

I probably need to start using feature branches to avoid showing every single PR from my fork in the history.

Roll Stress Interface:

Wigginns commented 1 year ago

Bug: Edit button for rich text fields is broken on all actors. I suspect it has to do with the consolidation of Listener functions.

[x] Fix Edit button for text fields

Wigginns commented 1 year ago

Example Delve with content.

Wigginns commented 1 year ago

It may be overkill to preserve the current user experience with Stress rolls. I'd be in favor of removing the option to choose type after the roll personally. Another option is to leave the stress type blank and pick it later