search

AutarchLLC / foundryvtt-acks-core

Adventurer Conqueror King System Imperial Imprint (ACKS II) core game module for the Foundry Virtual Tabletop.
https://autarch.co
Other
6 stars 2 forks source link

Update Character Sheet Displays: Add attribute for Initiative Modifier #20

Closed ArcanistWill closed 2 weeks ago

LeRatierBretonnien commented 1 month ago

Add also initiative/surprise bonus to monster sheets

ArcanistWill commented 3 weeks ago

@LeRatierBretonnien Is this where the sheet has been updated? image

NOTE - I think the calculation on the above area seems correct that it is showing a bonus. I don't think this is carried through to the Tweaks area. How are they supposed to be related? Compare: image

And: image

ArcanistWill commented 3 weeks ago

@Archon-Autarch If the initial attributes roll shows an Initiative bonus or penalty, is the Tweak supposed to be added to that to generate the final Initiative Bonus? See previous screenshots I attached.

LeRatierBretonnien commented 3 weeks ago

In fact, I forgot about this "tweak" window. So it is up to you to choose which one we keep 1 - We keep the new "Initiative bonus", and I drop the field in the Tweak window 2 - We keep the Tweak window, then I remove the new field I added , and I use the Tweak value in the Combat Tracker.

Tell me

ArcanistWill commented 3 weeks ago

For the Initiative Bonus, I want to ask @Archon-Autarch to confirm if he agrees with what I recommend, which is:

  1. Keep the calculated and displayed Initiative Bonus (screenshot 1)
  2. Keep the tweaks window function that will allow entering a "delta" Initiative Bonus (screenshot 2)
  3. (new) Calculate the final Initiative Bonus as the sum of the rolled bonus plus the tweak value entered.
  4. (new) Display the calculated final Initiative Bonus on the Abilities page (screenshot 3). Note that right now, I think the Abilities page displays only the Tweak value which looks like it overrides the calculated bonus?

That's my thought. Please confirm if this makes sense.

LeRatierBretonnien commented 3 weeks ago

OK, nor problem, let's wait confirmation from Archon-Autarch

ArcanistWill commented 2 weeks ago

I am testing latest build with all recent fixes (12.0.15) and confirm this can be closed.