League-of-Foundry-Developers / torch

Simple torch module for Foundry VTT
10 stars 16 forks source link

Every armor in Starfinder is also a light source #62

Open Beff42 opened 10 months ago

Beff42 commented 10 months ago

By default every armor in Starfinder has a flashlight attached. There are probably 100 or more suits of armor in the game. I could just add all of them into the source file, but I am not sure, if this is intended. Another possibility would be to extend the library.js to being able to understand a category or list of items, so all items of a certain type (here armor) can have the same light source. The default light configuration is currently serving as a workaround, but it's not really solving this, since not all actors wear armor.

I might be down to open a PR for either option above, just let me know which you prefer.

lupestro commented 10 months ago

There is the notion of an alias, which takes up less space than a full-fledged definition. Even so, adding that many would be daunting. I'll have to think on this one.