-
-
Notifications
You must be signed in to change notification settings - Fork 21.3k
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 ability to hide grid per viewport #88584
Add ability to hide grid per viewport #88584
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Huh, it makes sense in hindsight. Now it's a matter of thinking whether this is desired or not.
On a whim I'd say yes, and it doesn't look like this PR affects the existing, "global" toggle.
That menu is getting awfully tall though...
Fair. I personally use this feature along with #87793 and "View Gizmos" off to create a free flying game camera that can make micro adjustments for complex dynamic scenes while using a separate viewport that does have all of that stuff enabled to provide additional when context from a different angle when needed.
Agreed. Not as bad as the editor language menu though. Personally, I think some of the menus in the editor need to be reworked in general for reasons like in: godotengine/godot-proposals#8883 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm going to approve in principle, based on what I said before. There's nothing particularly odd about this PR. (still needs review from 3D editor team)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Didn't test, but the feature makes sense to me and the code looks straightforward.
56aff52
to
df43154
Compare
Thanks! |
Implements: godotengine/godot-proposals#9037
Related to: #87793
2024-02-20.01-29-16.mp4