This is a known issue with the asset system unfortunately, but I'll hopefully be able to address this before the 4.3 release. See #122439, #93726.
Closing as a duplicate/known issue. Thanks for…
Didn't confirm but I assume 656edfce18 broke this. But in general using button handling to activate the view item (for the context menu) seemed weak and turned out error prone. I'm much more…
This is a bit annoying, but wmOperatorCallParams
needs a pointer, and I rather stick to using that type. Maybe uiBut.opptr
should be a shared pointer, that might improve a few things actually.
Not sure, this is why I've also not done this yet. @HooglyBoogly was the "Weight" brush left in by accident and we should use the "Brush" one instead? CC @dfelinto
Pasting own answers from the chat:
Don't think the triagers have to take this list into account It's a lot easier to maintain a checklist than a bunch of tasks. I rather spend my time fixing…
I can't recreate this with main
, I think it's fixed with 4b168dd645?
But generally, we shouldn't have to create the "params" here, that should only be done in specific places as part of…
Can't recreate this (didn't try that hard), but I wonder if [this](https://pablovazquez.art/paste/#XQAAAQATCAAAAAAAAAAyGkkJwvwRfVDeCk9H2SRoLFgzPWJpxeINRgxFqCtNibsN+okSNS2BCMdY36XhNoG61ctbRPeaeC2QNs…