Scene Builder clobbers Terrain3D's hotkeys, causing data loss due to a confused undo stack

[sci-comp]

Please remove SceneBuilder while editing Terrain3D terrain for now. As it is, SceneBuilder will occasionally cause your edited work to not save.

[sci-comp]

#34

This commit likely did not fix this issue, more testing is needed.

[guicattani]

Which commit? This one? 7524bd6
I can take a look if you want... how do you reproduce the error?

[sci-comp]

Which commit? This one? 7524bd6 I can take a look if you want... how do you reproduce the error?

It was happening for me while creating a terrain with Terrain3D while Scene Builder is in the project. Just the ordinary sculpting and vertex painting operations. When I press Ctrl+Z, .... something isn't happening right. The undo may undo a ton of Terrain3D operations, instead of the last operation.

If you find any clues, please let me know! If #34 did fix this, then that's strange to me and I don't see why it would have, though #34 was a bug related to Undo management.

I don't need to work on terrain myself again anytime soon, so I've been procrastinating this one, haha. Good luck if you go hunting!

[sci-comp]

Oops, #34 is this issue, sorry. Yes, the commit you referenced is the one I mean.

Edit: Yes, this bug is still present. Somehow we're clobbering Terrain3D's hotkeys, which is confusing the undo stack.

[sci-comp]

I've added a "Disable Hotkeys" button, but the conflict persists even when this is toggled true.