Skip to content

Adjust transform plane positions based on camera orientation#120263

Open
Oblepikha wants to merge 1 commit into
godotengine:masterfrom
Oblepikha:better-transform-gizmo
Open

Adjust transform plane positions based on camera orientation#120263
Oblepikha wants to merge 1 commit into
godotengine:masterfrom
Oblepikha:better-transform-gizmo

Conversation

@Oblepikha

Copy link
Copy Markdown
Contributor

Fixes godotengine/godot-proposals#7358

before after
before.mp4
after.mp4

@Oblepikha
Oblepikha requested review from a team as code owners June 13, 2026 14:31
@Oblepikha
Oblepikha force-pushed the better-transform-gizmo branch 2 times, most recently from 7799666 to 5946958 Compare June 14, 2026 04:51
@Calinou Calinou added this to the 4.x milestone Jun 14, 2026
@Oblepikha

Copy link
Copy Markdown
Contributor Author

Fixed weird behavior when a gizmo is near the edge of the screen

@Oblepikha
Oblepikha force-pushed the better-transform-gizmo branch from 5946958 to d8c92a7 Compare June 14, 2026 06:54
@Oblepikha

Copy link
Copy Markdown
Contributor Author

Disabled in orthographic view. Do not move planes while a transform operation is in progress (dragging or scaling)

@ryevdokimov
ryevdokimov requested a review from a team June 14, 2026 17:13
@Oblepikha
Oblepikha force-pushed the better-transform-gizmo branch 4 times, most recently from 0fdf153 to 366a0e8 Compare June 14, 2026 18:44
@Oblepikha

Copy link
Copy Markdown
Contributor Author

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

before after
before.mp4
after.mp4

I also slightly increased the size of the transform planes. Hopefully no one has any objections 😅

@passivestar passivestar left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested, works as expected, UX makes sense, didn't look at the code

@Oblepikha
Oblepikha force-pushed the better-transform-gizmo branch from 366a0e8 to c96a0f8 Compare July 17, 2026 14:30
@Oblepikha

Copy link
Copy Markdown
Contributor Author

Merge conflicts resolved

@fire
fire requested a review from a team July 17, 2026 18:00

@YeldhamDev YeldhamDev left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please end the new comments with a period.

@Oblepikha
Oblepikha force-pushed the better-transform-gizmo branch from c96a0f8 to 6d58be4 Compare July 18, 2026 03:45
@Oblepikha

Copy link
Copy Markdown
Contributor Author

Updated the comments.

@Repiteo
Repiteo requested a review from YeldhamDev July 20, 2026 14:53
@Oblepikha
Oblepikha force-pushed the better-transform-gizmo branch 2 times, most recently from fbc9f0d to 44b2b32 Compare July 23, 2026 04:21
@Oblepikha

Oblepikha commented Jul 23, 2026

Copy link
Copy Markdown
Contributor Author

Implemented the center controls proposed in godot-proposals/issues/14563
image

@Oblepikha
Oblepikha force-pushed the better-transform-gizmo branch from 44b2b32 to 7dec399 Compare July 23, 2026 04:38
@Mickeon

Mickeon commented Jul 23, 2026

Copy link
Copy Markdown
Member

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.

@Oblepikha
Oblepikha force-pushed the better-transform-gizmo branch from 7dec399 to 22e4ca9 Compare July 23, 2026 14:12
@Oblepikha
Oblepikha force-pushed the better-transform-gizmo branch from 22e4ca9 to a81ddee Compare July 23, 2026 14:18
@Oblepikha

Copy link
Copy Markdown
Contributor Author

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Orient the 3D manipulation gizmo to be easy to use regardless of camera rotation

6 participants