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,…
I'm planning on extracting the bpy_types.py and wm.py changes to a separate PR / commit, since it's more core functionality and shouldn't be bundled with this, but I think the context of the…
I don't know if there's a more appropriate way of achieving what this line does - without this, after Shift-S is pressed, the value in the header does not update until the cursor moves over the toolbar or outside of Blender's window.
