fvtt-fria-ligan / morkborg-foundry-vtt

An unoffical, community-supported system for playing MÖRK BORG on Foundry VTT.
Other
33 stars 17 forks source link

Make Powers per day text interactive like Omens per day #148

Closed Lim-Dul closed 2 years ago

Lim-Dul commented 2 years ago

Expected Result Clicking the Powers per day info text triggers a macro resetting usage similar to the Omens per rest info text.

Actual Result Hovering over the Powers per day info text highlights it as if it was an interactive element but clicking it does nothing. The only way to reset power usage is per the rest button or manually. This is inconsistent with how a similar functionality for Omens works, where you can click the info text. image image

Steps to Reproduce

  1. Open a player's character sheet
  2. Click on the "Powers" tab
  3. Hover over the "PRE+d4 times per day" info text and attempt clicking on it
mcglincy commented 2 years ago

There's a rollable span in character-sheet.html: < span class="rule-text rollable powers-per-day-text" >{{localize "MB.PowersPerDayText"}} < /span >

but character-sheet.js activateListeners() doesn't actually hook anything to it. There is already a _onPowersPerDayRoll() method defined, so should be just adding the click handler.

Lim-Dul commented 2 years ago

Confirmed fixed in v1.14.0.