Skip to content

fix(options): Fix System Time and Game Time settings not loading and applying correctly#1349

Merged
xezon merged 1 commit into
TheSuperHackers:mainfrom
Mauller:fixup-timers-settings
Jul 27, 2025
Merged

fix(options): Fix System Time and Game Time settings not loading and applying correctly#1349
xezon merged 1 commit into
TheSuperHackers:mainfrom
Mauller:fixup-timers-settings

Conversation

@Mauller

@Mauller Mauller commented Jul 23, 2025

Copy link
Copy Markdown

This PR is a small fixup pr in which the value zero was not being considered when saving the game and sytem time font sizes in the game options.

Zero is used to disabled the game time and will eventually be input by the user through the options menu.

@Mauller Mauller self-assigned this Jul 23, 2025
@Mauller Mauller added Minor Severity: Minor < Major < Critical < Blocker Gen Relates to Generals ZH Relates to Zero Hour ThisProject The issue was introduced by this project, or this task is specific to this project labels Jul 23, 2025
Comment thread Generals/Code/GameEngine/Source/GameClient/GUI/GUICallbacks/Menus/OptionsMenu.cpp Outdated
Comment thread Generals/Code/GameEngine/Source/GameClient/GUI/GUICallbacks/Menus/OptionsMenu.cpp Outdated
@xezon

xezon commented Jul 23, 2025

Copy link
Copy Markdown

Unclear what "OSD" means. Prefer not using acronyms.

@ElTioRata

ElTioRata commented Jul 23, 2025

Copy link
Copy Markdown

OSD = On Screen Display

@Mauller Mauller force-pushed the fixup-timers-settings branch from 0607b78 to f099eba Compare July 24, 2025 18:30
@Mauller

Mauller commented Jul 24, 2025

Copy link
Copy Markdown
Author

This should be good to go now.

@Mauller Mauller changed the title [GEN][ZH] Fixup zero not being considered when saving OSD font sizes [GEN][ZH] Fixup system and game time settings not considering zero or being loaded from ini Jul 24, 2025
@xezon xezon changed the title [GEN][ZH] Fixup system and game time settings not considering zero or being loaded from ini fix(options): Fixup system and game time settings not considering zero or being loaded from ini Jul 27, 2025
@xezon xezon changed the title fix(options): Fixup system and game time settings not considering zero or being loaded from ini fix(options): Fix system and game time settings not considering zero or being loaded from ini Jul 27, 2025
@xezon xezon changed the title fix(options): Fix system and game time settings not considering zero or being loaded from ini fix(options): Fix System Time and Game Time settings not loading and applying correctly Jul 27, 2025
@xezon xezon merged commit ebe14da into TheSuperHackers:main Jul 27, 2025
19 checks passed
@xezon xezon deleted the fixup-timers-settings branch July 27, 2025 08:21
fbraz3 pushed a commit to fbraz3/GeneralsX that referenced this pull request Nov 10, 2025
fbraz3 pushed a commit to fbraz3/GeneralsX that referenced this pull request Feb 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Gen Relates to Generals Minor Severity: Minor < Major < Critical < Blocker ThisProject The issue was introduced by this project, or this task is specific to this project ZH Relates to Zero Hour

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants