- Issue created by @mpractic
- First commit to issue fork.
- 🇨🇦Canada mandclu
The code in the MR seems to work, but with one limitation: If you go into Format Settings > Event Colors and select a taxonomy field and vocabulary, it will do an AJAX refresh and show a color input for each term in the specified vocabulary. Setting a value without saving first won't work, however. You need to set the vocabulary, save the settings, and then reopen the settings. Now any changes made will be saved and should be reflected immediately in the Preview, and on the normal view display after changes are saved.
This seems to be a limitation of the Form API, since elements added after the form's creation are excluded from the submitted data for security reasons. I have considered instead of immediately showing elements that won't be saved, showing instead a message to save the settings and reopen them in order to assign colors.
Marking this "Needs work" until we can figure out the UX issue, but also open to any other feedback on how this could be improved.
- 🇨🇦Canada mandclu
I decided to go ahead and change the UX to show a message after selecting a vocabulary, so that nothing is promised that isn't functional. With could commit this and then file a separate issue to try and show the term color inputs immediately.
- 🇨🇦Canada mandclu
I decided what's been implemented so far is fine as a first step for a late alpha/early beta release, since we can continue to iterate on it if more work is needed. I will also create the follow-up issue to show the color fields immediately.
Automatically closed - issue fixed for 2 weeks with no activity.