Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add option to toggle visibility of Position gizmos in 2D editor, organize existing options #75005

Merged
merged 1 commit into from
Jan 4, 2024

Conversation

J-N-Witch
Copy link
Contributor

@J-N-Witch J-N-Witch commented Mar 16, 2023

This change lets you hide Position gizmos on 2D nodes
allowing you to make clean screenshots in the editor without crosses everywhere, among other things.

In order to avoid clutter in 2D > View menu, "Show Group And Lock Icons" and "Show Gizmos"
were moved to "Gizmos" submenu along with the newly added option.

Before:

4

After:

menu.mp4

Related proposal: godotengine/godot-proposals#6344

show_edit_locks (which actually referred to showing locks and groups)
was renamed to show_lock_gizmos to be consistent with the existing show_transformation_gizmos;
and was decoupled from group gizmos.

I would like have this in 3.x as well.

@Rubonnek Rubonnek added this to the 4.1 milestone Mar 16, 2023
@YuriSizov YuriSizov modified the milestones: 4.1, 4.2 Jun 14, 2023
@Drkie
Copy link

Drkie commented Oct 4, 2023

@YuriSizov any reason this got moved from 4.1? It doesn't seem like it will make it to 4.2 either.

@YuriSizov
Copy link
Contributor

YuriSizov commented Oct 4, 2023

@Drkie Nobody reviewed it thus far, so it can't be considered for the next release. Indeed, it won't make the cut for 4.2 either at this point, because we're entering feature freeze. But thanks for bumping it, that should put more eyes on it 👍

@YuriSizov YuriSizov modified the milestones: 4.2, 4.3 Oct 4, 2023
@YuriSizov YuriSizov requested review from a team October 4, 2023 21:15
@YuriSizov YuriSizov requested a review from a team October 4, 2023 21:16
@KoBeWi
Copy link
Member

KoBeWi commented Oct 9, 2023

IMO a single-element separated section is a bit awkward:
image
Maybe the Gizmos could be part of the first section. You could move the Grid to the bottom of the section, so two menus would be next to each other. Also Clear Guides is at odd spot, it could be near the checkboxes too (though not sure).

@J-N-Witch
Copy link
Contributor Author

IMO a single-element separated section is a bit awkward

This is exactly how the existing Gizmos menu in 3D looks like.

3d-gizmos

The pattern is also repeated in many other places. See "Scene > Export As...", "Project > Version Control" etc.

Anyways, does this look better or should we keep the current layout?

2d-gizmos-updated

Also Clear Guides is at odd spot, it could be near the checkboxes too (though not sure).

I think messing with Clear Guides is out of scope for this change.

…editor

instead of standalone checkboxes like "Show Group And Lock Gizmos".

Add option to toggle visibility of Position gizmos.
@J-N-Witch J-N-Witch force-pushed the add-position-gizmo-toggle branch from 5af18b2 to d832f62 Compare October 25, 2023 13:05
@J-N-Witch
Copy link
Contributor Author

Any thoughts on my comment above @AThousandShips @KoBeWi ?

I have rebased the change on the latest master and implemented your suggestions, let me know if there's anything else I can do.

Copy link
Member

@AThousandShips AThousandShips left a comment

Choose a reason for hiding this comment

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

Haven't tested but code looks good

@akien-mga akien-mga merged commit 94f72a7 into godotengine:master Jan 4, 2024
15 checks passed
@akien-mga
Copy link
Member

Thanks! And congrats for your first merged Godot contribution 🎉

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.

7 participants