Yeah, BLI_assert_unreachable is weird in that context and probably not worth considering at all. I thought of using it instead of BLI_assert(0) since
if (mesh_eval == nullptr)
{
…I saw there was also discussion about the usefulness of the assert in the chat - I get the distinction between developer-facing informative asserts & things done to prevent crashes, but the…
After doing a bit of investigation, I believe this isn't possible currently for the same reasons that the Shift-S keybind isn't possible:
Classes that subclass Panel have a `draw(self,…
As far as I'm aware, the only other case that has the options displayed in the toolbar in this same way is the Annotation tool. I think it would be fine to group these under a "Stroke" popover to…
This doesn't appear to be exclusive to Sculpt Mode, I see the same flickering for Vertex Paint, Weight Paint, etc
Appears to work the same in Edit mode and Sculpt mode now, no code comments from me, though I agree with Iliya and Hans on the constexpr