BlitzKraig / fvtt-CommunityLighting

FoundryVTT Community Lighting Animations for 0.7.x+
MIT License
9 stars 6 forks source link

Use optgroup for grouping lighting entries #4

Closed rmasoni closed 3 years ago

rmasoni commented 3 years ago

I couldn't find where this was set up in the code, but the “Light Animation Type” select should use optgroup to group the options by author — it's the HTML standard and most appropriate element for this.

There's no nesting for optgroup though, so it's not possible to put everything inside a parent “Community Lighting” group. My suggestion is to use Community Lighting (Author), like this:

image

Instead of:

image
BlitzKraig commented 3 years ago

Hi,The animation selection is foundry core functionality. I do plan on doing something with it, the current solution is just a workaround.

BlitzKraig commented 3 years ago

Hi rmasoni, I'm working on some features that manipulate the form for other reasons at the moment, so I think I'm going to try and add in the optgroup for next release too. Thanks for the suggestion!

rmasoni commented 3 years ago

Cool, thanks!