League-of-Foundry-Developers / torch

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

Updates to simplify and clarify defining data for light sources #31

Closed lupestro closed 1 year ago

lupestro commented 1 year ago

The work here is intended to address the difficulties users have been encountering creating the JSON for light sources, as discussed at length in issues #23, #27, and #29.

lupestro commented 1 year ago

With a little more thought and a fifteen minute walk, I realized that the aliases need to be newValue: oldValue, not oldValue: newValue. I had originally thought of the latter as comparable to translation key: value files. However, suppose that the game needs multiple aliases for the same original value. Doing them separately with the current syntax, we'd end up with duplicate keys. We could make the value an array of new names, but it would be much simpler to turn it around, so that we define the target for each new name separately. It's also much more intuitive for the user, I think, to have the key be the term you're defining, and the value being what it refers to.