While this works well, the relationship between the cursor motion & the lamp radius is off, although I think this relates to the gizmos modal function - so it could be handled separately.
I like most of the idea.
My only remark is that core functionality that happens to be implemented as add-ons (I/O mostly, but also OpenXR and Grease Pencil features) should be brought to…
Using events from the gizmo group seems like it could back-fire in the future, see longer reply with alternative solution.
While the functionality works well, this is handling events from the gizmo-group which may conflict with the gizmos event handling (in the future, even though it works now), as it means changes to…
Turns out I was too quick to dismiss the approach from this patch, while checking for *
isn't right, fnmatch
is quite simple so a str.startswith
check is valid as long as the string doesn't…
As noted in reply, this will break support for some fnmatch
features.
This change only supports checks for the *
character, fnmatch
also supports characters within brackets & ?
, so I don't think these kinds of optimizations are practical.
However 400ms for…
Thanks, committed 5d30c3994e6ecf665bfae87d7294216d368f0c59, closing.