Differential D8928
Outliner: Move collection exclude icon to restrict column Authored by Nathan Craddock (natecraddock) on Sep 17 2020, 5:28 PM.
Details Move the collection exclude icon right to be with the other restrict  This also has the benefit of completely removing some duplicated code used for drawing the checkbox.
Diff Detail
Event TimelineComment Actions +1 I personally did not like this design when I first saw it because enable/disable affects the collection and its hierarchy in a way that the other restriction columns don't (removes objects from depsgraph, even removes hierarchy lines and restriction columns underneath). However! Actually seeing this in context with large files, hierarchy is way more clear now with all objects indented under the collection name. This improves readability in large files. One note: when the collection is disabled (checkbox ticked off), shouldn't the restriction columns be faded? Just like the collection name and icon. Comment Actions Personally I also like this change. We already discussed it with Dalai, and he's fine with it. The only suggestion he had was separating it better visually from the other icons. Maybe we could draw a vertical line? It is a higher level switch so it was intentional to keep it on the left. But with the new patch the hierarchy becomes more clear, which is (arguably) the more important thing to communicate visually. Comment Actions I think adding a vertical line would just add visual noise. As far as users are concerned, "exclude" is just another RTO. Comment Actions I don't feel strongly about this either way, if the other members of the UI team prefer it then it's fine with me. |