greedyj4ck / DEGENESIS-FoundryVTT

DEGENESIS: Rebirth system for Foundry Virtual Tabletop
26 stars 22 forks source link

Initiative from Combat Tracker #13

Closed KristjanLaane closed 3 years ago

KristjanLaane commented 3 years ago

Initiative integration when rolled directly from the Combat Tracker. Option to also show number of actions available based on how many triggers were rolled - use fighting.actionsCurrent for the Tracked Resource under Combat Tracker Settings.

CombatTrackerInitiative

moo-man commented 3 years ago

Isn't there a process by which you spend a certain amount of ego?

KristjanLaane commented 3 years ago

Yes, spending ego points is not implemented here, but it's optional to spend ego points and can only be done on the first round. The use case here is if you have a number of characters in combat and you want to quickly roll all their initiatives each round - you can click Roll All and the rest is magic. I implemented it because I need it as I don't want to slow the fights down by individually rolling lots of initiatives, and do that again individually for each character involved every single round :)

moo-man commented 3 years ago

Sure, I understand, just considering how to handle the ego spending in the future.

moo-man commented 3 years ago

my only issue here is the very basic chat card this produces. image

I've added synchronous roll functions so that initiative rolls can go through the same rendering process as normal rolls. I'm not very happy with this solution as it's duplicated code and redundant chat cards (the basic chat card is still displayed). But I'm open to thoughts.

KristjanLaane commented 3 years ago

After some digging found a way to remove the basic chat card and replaced it completely with a Degenesis-style one.

Other things, too: Initiative calculation rolls spent ego dice as well now - successes from the extra dice, but not triggers. Initiative from the Combat tab in Actor sheet now triggers the fully-fledged initiative response, same as from the Combat Tracker.

Please note that due to template changes, most of this does not work with characters created before this commit - create new actors to use this stuff.

Also, Combat Tracker tracked resource changed to state.initiative.actions - to get that number of actions to show as the secondary value in the Combat Tracker.

moo-man commented 3 years ago

do you have a discord? I'd like to talk about this more.

moo-man commented 3 years ago

Going to go ahead and merge this, feel free to make further changes in another PR if you want.