Kristian Duske

Looks good, thanks for addressing my comments. I'll merge once all runs are green.

> This overall sounds great. I have some questions about the refactoring. > I'm not seeing much overlap with the rest of this feature (the refactoring is mostly about unifying...

@ericwa I'll start working on the refactoring because I need that for the Bezier patch control points tool(s). With this refactoring, I hope to be able to reuse a lot...

I think this is overcomplicated. A simple option to disable autosave altogether from the preferences should be enough.

@ericwa I can reproduce this under macOS. Could you give it a try on Windows?

Thanks for checking! I can't really tell why it's behaving like this on macOS, but I assume it has to do with how events are processed in Qt on macOS....

No problem!

This issue is not about separate floating windows for 2D,3D views. These will always be in one window.

Can you provide a pull request?

I'm not sure if we really want to change it so that TB leaves the face mode when you shift click a face. That could disrupt other people's workflows.