- Amsterdam, Netherlands
- https://cessen.com
-
Animator, rigger, and software developer. Currently working at the Blender Institute as a developer on Blender's animation system.
Been using Blender since 1998, and worked on Big Buck Bunny and Sintel (two of Blender's open movie projects).
- Joined on
2003-03-21
Seems about as good as we can do, I think. Only one minor nit: instead of "into", just "in" sounds more natural here.
The path field isn't shown without a configured datablock either. Thus the new UI follows the current behavior in my view.
Ah, that's fair. Let's leave it as you have it, then. If/when we…
Yeah, this seems better to me. Although it did make me realize even more how sprawling and scattered throughout the source code the logic for drivers is in general.
A nit that I don't feel…
You changed it to "The Only Show Errors filter should include drivers that...", which reads like a recommendation, not a description. I think it's better to either stick with what you had, or…
I'm struggling a bit to come up with something better, but maybe this reads a little better:
Just a few more nits, but otherwise the code looks good to me.
If I understand correctly, this isn't a setting but rather an indicator of what happened. Is that correct? If so, I think this might be a bit clearer as a description: