Adjust transform plane positions based on camera orientation#120263
Adjust transform plane positions based on camera orientation#120263Oblepikha wants to merge 1 commit into
Conversation
7799666 to
5946958
Compare
|
Fixed weird behavior when a gizmo is near the edge of the screen |
5946958 to
d8c92a7
Compare
|
Disabled in orthographic view. Do not move planes while a transform operation is in progress (dragging or scaling) |
0fdf153 to
366a0e8
Compare
|
I rewrote the way move and scale handles are selected. They are now approximated with spheres, projected onto the screen, and evaluated based on their distance to the mouse cursor. This makes it possible to select occluded handles
I also slightly increased the size of the transform planes. Hopefully no one has any objections 😅 |
passivestar
left a comment
There was a problem hiding this comment.
Tested, works as expected, UX makes sense, didn't look at the code
366a0e8 to
c96a0f8
Compare
|
Merge conflicts resolved |
YeldhamDev
left a comment
There was a problem hiding this comment.
Please end the new comments with a period.
c96a0f8 to
6d58be4
Compare
|
Updated the comments. |
fbc9f0d to
44b2b32
Compare
44b2b32 to
7dec399
Compare
|
Hold on. The PR is going way out-of-scope of what was originally intended. I would recommend sticking to one thing at the time, and opening separate PRs for each feature/fix. The PR's title and commit name do not even match what is being done anymore. Please just stick to what was approved of last time. |
7dec399 to
22e4ca9
Compare
22e4ca9 to
a81ddee
Compare
|
Yeah, fair point. I started working on it and got carried away. I've reverted the latest changes and moved them into a separate PR |

Fixes godotengine/godot-proposals#7358
before.mp4
after.mp4