[UX] Show a message to the user suggesting disabling verbose logs#5452
Open
[UX] Show a message to the user suggesting disabling verbose logs#5452
Conversation
arielj
requested review from
a team,
Etaash-mathamsetty and
flavioislima
and removed request for
a team
arielj
commented
Apr 2, 2026
| 'Looks like the game is working. You might want to disable verbose logs for better performance. Click the ^ icon and use the "Play Now" button without logs' | ||
| )} | ||
| </span> | ||
| )} |
Collaborator
Author
There was a problem hiding this comment.
the only change in this return value is this added element above .playButtons, the rest did't change, it's just the diff messed up because of the updated indentation adding a <></> wrapper
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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 might be a solution for #5134
Currently, we have logs enabled by default to make support and troubleshooting easier, but when a game is already working correctly there's no real need to keep the verbose logs enabled and for some games with too much output this can really hurt performance.
This PR adds a message next to the play button suggesting the user to disable logs if they are enabled and the total played time is more than 5 minutes (I imagine that if there's a 5 minutes play time it means the game runs, which I think it's good enough).
This way, users can see the message and turn off the verbose logs and also learn about this and the possible impact.
Use the following Checklist if you have changed something on the Backend or Frontend: