Skip to content

Comments

Hide more advanced video options#302

Merged
wookieejedi merged 4 commits intoKnossosNET:mainfrom
wookieejedi:cleanup-video-options
Feb 10, 2025
Merged

Hide more advanced video options#302
wookieejedi merged 4 commits intoKnossosNET:mainfrom
wookieejedi:cleanup-video-options

Conversation

@wookieejedi
Copy link
Contributor

There have been new players/testers that have disabled Vsync and 120 FPS Limit without fully realizing the downstream side effects and then ran into various issues. These are typically somewhat advanced options for graphics, so after talking it over this PR removes them by commenting out the UI fields from the global graphics options.

There have been new players/testers that have disabled `Vsync` and `120 FPS Limit` without fully realizing the downstream side effects and then ran into various issues. These are typically somewhat advanced options for graphics, so after talking it over this PR removes them by commenting out the UI fields from the global graphics options.
@wookieejedi wookieejedi added the cleanup Cleanup code or other aspect label Feb 9, 2025
@wookieejedi wookieejedi added this to the 1.3 milestone Feb 9, 2025
@wookieejedi wookieejedi merged commit 2f3e191 into KnossosNET:main Feb 10, 2025
@wookieejedi wookieejedi deleted the cleanup-video-options branch February 10, 2025 19:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cleanup Cleanup code or other aspect

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants