Yeah, I think that's a good argument, that all_verts
is a rarer usecase and should stick out more
One comment on the naming, otherwise looks good to me.
I personally think unique_verts
and all_verts
is better than implying one or the other (e.g. unique_verts
and verts
or verts
and all_verts
), but I don't feel too strongly about this at the moment.
One of the bad commits is 7e0029fb656a8307ad9a8cb18f097f0483599a28. though this causes the entire mesh to disappear when doing any stroke.
For now I'll mark this as confirmed, since even in 4.3 I can see the potential for this bug to surface; the current way of trying to prevent the brush action from changing mid-stroke isn't very…
I'm unable to reproduce this either, on Linux with a Huion H950P
Behavior here seems sane to me in testing both the provided mesh in the bug report and on a default cube. I think I'd still prefer if @JulienKaspar took a look as well before merging this in case there is some reason we would want to keep the old "Relative Mode" or if this behaves weirdly for some other usecases that I'm unaware of.