Condition Clusters
Stanzilla opened this issue · 7 comments
As a continuation of #957
I really dislike how some of our on/off Conditions look at the moment
That lonely checkbox really needs some label.
I could not find where I proposed it first, it might have been just on Discord, but I really think we should move to a cluster of options for things that can be clustered like Glow
, Offsets
, cooldown like we already do on the Display
tab
This would end up looking something like this:
Problems with our current (newish) system of two dropdowns:
- too many dropdowns
- if you want to change two things about the first selection, you have to duplicate it, ending up with two "Glow" or whatever dropdowns
Visibility
is still a lonely checkbox with no label
I think while the condition ui is not 100% ideal, the orthogonality of the various glow settings is now common knowledge and I think this is no longer something worth keeping open.
The most common use case of glows imho is to configure it once in the main settings and then toggling it on/off via a condition. I don't think having to configure all those things every time a glow is toggled is good.
Well you would not have to, you just could. If we use the collapse design + the advanced button design you made, the main option could just be on/off and when users click on advanced they get the full sheet.
Also this is not only about glow.
The collpased design doesn't realy solve that, because we would need a additional toggle for "apply color change" in addition to "color".
Oh thanks for adding that to the ticket, I totally forgot to do that after our discussion
We could do this, but in order for it to not be a loss of functionality, these “clusters” would need to be able to express that certain values aren’t specified, and shouldn’t be modified on this property change. The same concept as inherited attributes in css, I suppose.
I do think the biggest offender is glow like infus mentioned. If rivers solution of inherited attributes is workable and simple enough I think that would be greatly useful moving forward. If it's not though we could just work through the display tab.
Right now most people just select "add glow" and move on to the conditions. Within the properties there we could let users set all the defaults of the "12", yes twelve, properties for glow activity. Essentially letting users preset the properties. Then when they select Glow -> Use Preset it uses all the things they set.
As part of the larger discussion I do think the drop downs are really important for this particular context of UX/UI. The reasoning is conditions form a series of logic chains and by selecting the dropdowns you're parsing and working through that chain in an organized manner. I think this really helps the novice/moderate users a lot. If we throw all the options on the screen like much of our existing display we're going to get a bit of overload and eye fatigue. That's something I've been working on in the background and I have some stuff I've shown stan and ipse. Maybe it's time I brought it out even though it's not quite finished.