Skip to content

Simply settings window usage#1504

Merged
chreden merged 2 commits intomasterfrom
issue/1503
Oct 4, 2025
Merged

Simply settings window usage#1504
chreden merged 2 commits intomasterfrom
issue/1503

Conversation

@chreden
Copy link
Owner

@chreden chreden commented Oct 3, 2025

Just pass the settings object to the settings window instead of passing the individual settings in.
Now only one event raised in most cases since it was only really ever forwarded and then set in the settings object.
Only exception at the moment are linear filtering and fonts.
Closes #1503

Just pass the settings object to the settings window instead of passing the individual settings in. Now only one event raised in most cases since it was only really ever forwarded and then set in the settings object.
Only exception at the moment are linear filtering and fonts.
Closes #1503
@chreden chreden added this to the Next milestone Oct 3, 2025
@chreden chreden requested review from lahm86 and makotocchi October 3, 2025 21:55
@chreden chreden self-assigned this Oct 3, 2025
@chreden chreden enabled auto-merge (squash) October 3, 2025 21:55
@chreden chreden merged commit 6d14022 into master Oct 4, 2025
1 check passed
@chreden chreden deleted the issue/1503 branch October 4, 2025 08:23
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.

Make SettingsWindow easier to change or add to

2 participants