-
Notifications
You must be signed in to change notification settings - Fork 740
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
Split interaction menu settings between interaction and self interaction #6609
Split interaction menu settings between interaction and self interaction #6609
Conversation
not sure which way is wanted here, but I might as well be consistent in my own PR
Is there anything outstanding I need to do on this PR? It should it is good for a full review. |
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.
Some minor formatting changes
I changed the default of cursorKeepCenteredSelfInteraction to false (how it was before) |
Might be fixed soon:tm:, thanks to @dedmen |
This merge broke #7381 |
When merged this pull request will:
This is something that's always annoyed me about self interaction, so I figured I'd finally do something about it. Pretty minor but personally not seeing people wave their guns in the air as they navigate self interaction is a decent improvement.
Edit note: By default both backgrounds are set to disabled, my personal setting is the blur screen.