foundryvtt / pf2e

A community contributed game system for Pathfinder Second Edition.
https://foundryvtt.com/packages/pf2e
Apache License 2.0
410 stars 344 forks source link

Include damage predicates when predicating roll notes for damage rolls #14938

Open MrVauxs opened 4 months ago

MrVauxs commented 4 months ago

It would be great to create Rule Elements that predicate upon dealing damage of a specific type. For example, creating a Note for everytime you roll vitality damage.

There seems to already exist a "damage:type:vitality" string in the code, but it does not work either as a selector or predicate.

stwlam commented 4 months ago

Remove the predicate.

CarlosFdez commented 4 months ago

Does DamageAlteration suffice to increase already existing vitality damage? Would an upgrade there help with that?

MrVauxs commented 4 months ago

It was discussed on discord that having general rule options for damage types would be beneficial for other effects.

By this point I just want to create a Note RE that triggers whenever you roll any amount of vitality damage.

Altering Damage wont help here for my specific use case (unless the system is capable of only doing that for half that damage to another type, so 50% Vitality 50% Fire for example).

stwlam commented 4 months ago

At this point I really don't understand what you're asking for

MrVauxs commented 4 months ago

A way to predicate on rolling vitality damage. Ex. damage:type:vitality becoming a selector or predicate that you can actually use, as its apparently created in code already but never added to Roll Options or any actionable elements. image

CarlosFdez commented 4 months ago

From discord discussion, this is a case of not being clear. Vauxs seems to want to spawn a note in damage rolls when certain types of damage occur, and would like the modifier specific roll options to be available for note predication.

MrVauxs commented 4 months ago

I will note that this also applies to inline damage rolls. Despite being able to roll for example @Damage[1[fire]], it does not give any information on it dealing fire damage.