I can spin off a separate design task for the points you've listed here. I appreciate the feedback on the next steps for it. For this to be in a shippable state, do you think the only areas that…
This is needed inside draw_lasso_smooth_stroke_indicator for the current mouse position, as we don't have a handle to the wmEvent in the wm_gesture_draw_* methods to get the current mouse…
When I was doing some testing, this only seemed to occur specifically for the new use_smooth_stroke property on the operator. I couldn't replicate this for a handful of WM_OT_context_toggle…
Only just noticed that the build failed, Looking into the test failures now...
Overall I agree with the changes and I think this looks much better organized as a whole, though I can't really provide detailed input from the user side. Requesting changes specifically for the line in space_view3d.py
This name is slightly odd to me. I see that showhide has a precedence elsewhere in this file, but I think this is probably fine as VIEW3D_MT_sculpt_trim
2, Line Mask is missing from Mask menus gesture operators. Is that intentional or forgotten?
Found the [commit](https://projects.blender.org/blender/blender/commit/8c81b3fb8b9c857626c037fe0c95…
Why remove this? It doesn't hurt AFAIK
It doesn't hurt, but it doesn't really provide any benefit either to how we use the value in the code at the moment, so I view it as kind of misleading,…