feat(ui): Implement simple Latency and FPS display #1546
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Caball009
added
Enhancement
xezon
reviewed
Sep 6, 2025
L3-M
force-pushed
the
show-latency-fps
branch
from
3b1cc70 to
6bf33ea
Compare
|
Positioning code simplified. |
L3-M
force-pushed
the
show-latency-fps
branch
from
6bf33ea to
2f8d2d6
Compare
xezon
reviewed
Sep 7, 2025
GeneralsMD/Code/GameEngine/Source/GameClient/GUI/GUICallbacks/Menus/OptionsMenu.cpp
Outdated
Show resolved
Hide resolved
L3-M
force-pushed
the
show-latency-fps
branch
from
2f8d2d6 to
718196b
Compare
|
Applied all suggestions. |
34 tasks
L3-M
force-pushed
the
show-latency-fps
branch
from
718196b to
f8be888
Compare
|
Addressed all comments and rebased onto main. |
xezon
reviewed
Sep 25, 2025
L3-M
force-pushed
the
show-latency-fps
branch
from
f8be888 to
3ba7b00
Compare
|
Tweaked. |
xezon
reviewed
Sep 25, 2025
L3-M
force-pushed
the
show-latency-fps
branch
from
3ba7b00 to
352a25e
Compare
xezon
reviewed
Sep 25, 2025
L3-M
force-pushed
the
show-latency-fps
branch
from
352a25e to
d146644
Compare
xezon
reviewed
Sep 26, 2025
L3-M
force-pushed
the
show-latency-fps
branch
2 times, most recently
from
a4d64cb to
8dfbe45
Compare
xezon
reviewed
Sep 27, 2025
L3-M
force-pushed
the
show-latency-fps
branch
from
8dfbe45 to
98b1454
Compare
|
Tweaked. |
|
Replicated to Generals. |
L3-M
force-pushed
the
show-latency-fps
branch
from
e80c8f6 to
13dd79c
Compare
L3-M
force-pushed
the
show-latency-fps
branch
from
13dd79c to
8cfddf6
Compare
xezon
changed the title
|
I thought this could be disabled but setting all 4 values to 0 just crashes the game. 1 is the lowest value. |
|
Please create bug report. |
Looks like to crash only when |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This change implements Latency Counter (left), FPS Counter (left), and FPS Limit (left) in Generals and Zero Hour.
They are enabled by default with a size of 8 and positioned before (to the left of) SystemTime, replicating GenTool implementation.
The following options have been added:
They can be styled and repositioned by adding the relevant fields to the InGameUI.ini data file.
Example:
In a Network Game:
In Menus:
TODO