Skip to content

Comments

Update default values#270

Merged
wookieejedi merged 1 commit intoKnossosNET:mainfrom
wookieejedi:default-tuneups
Jan 22, 2025
Merged

Update default values#270
wookieejedi merged 1 commit intoKnossosNET:mainfrom
wookieejedi:default-tuneups

Conversation

@wookieejedi
Copy link
Contributor

Very small PR that changes 2 default settings.

  1. Makes the default Window Mode Windowed instead of Fullscreen. The help text already recommends putting it at Windowed mode so it makes more sense to make that the default.

  2. Turns off TTS by default to better match the newer defaults of this setting in FSO. Happy to discuss alternatives if desired, such as keeping TTS mode enabled by default but having the specific settings only have it on in-mission by default, or something else. Overall, it's just very strange to new players to startup the game and have things being read to them right away as they navigate around the techroom and mission briefings, etc.

Very small PR that changes 2 default settings.

1) Makes the default Window Mode `Windowed` instead of `Fullscreen`. The help text already recommends putting it at `Windowed` mode so it makes more sense to make that the default.

2) Turns off TTS by default to better match the newer defaults of this setting in FSO. Happy to discuss alternatives if desired, such as keeping TTS mode enabled by default but having the specific settings only have it on in-mission by default, or something else. Overall, it's just very strange to new players to startup the game and have things being read to them right away as they navigate around the techroom and mission briefings, etc.
@wookieejedi wookieejedi added the cleanup Cleanup code or other aspect label Jan 21, 2025
@wookieejedi wookieejedi merged commit 895bd87 into KnossosNET:main Jan 22, 2025
@wookieejedi wookieejedi deleted the default-tuneups branch January 22, 2025 00:39
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