Repository navigation
Conversation
From a UX review of the whole window. Text, icons, order, window size and
keyboard only: what a run does, how it is stopped and what is put back are
unchanged.
- Before and during a run the window says the screen can go black for a
moment and that the PC must not be switched off; the result page no
longer is the first place that says so.
- Abort is an X with "Abort run", outlined red, and says "Stopping and
putting the card back..." once clicked. The title row shows how far the
run is ("12 of about 40 min"); a short step counts down as "5 s left".
- Without a saved tune the Dashboard's blue button is "Optimize GPU...";
without administrator rights the Optimize page keeps its Optimize button,
and the elevated copy opens on that page with the profile still selected
(--open-optimize=N).
- A run the user stopped or did not start reads "Stopped", not a red
failure. Other failures get a capitalised reason and where run.log is.
The GUI never tells a user to run `gao --reset`; boot.log and the command
line keep the engine's words, the Dashboard log says them plainly.
- Result pages: "Done: 4.6 % faster, 6 % more power", "Speed (stress
test)", the power change in percent for every profile, and [Done] first
with Revert to stock apart at the right.
- Dashboard: three log lines by default, an icon button for Detect GPU, a
next step under "Set by another program", "power limit" spelled out.
- Fan page: a short explanation, "Back to the tested curve", and a button
in both empty states. About text brought up to date.
- Tray: Exit during a run asks first; Revert and Re-apply from the tray say
what they did in a balloon while the window is hidden; failures use the
warning icon; a driver update offers the 5-minute test, as at logon.
- Window: a minimum size of 1000x700 (scaled), opened inside the monitor's
work area, wrapped text and card titles that shrink instead of being
cut, the scrollbar's room always kept, and Tab/arrows/Enter/Space work
(ImGui keyboard navigation).
- Consistent terms and units: "Don't start" on the question about other
programs, "Not administrator", "Details", degrees as "°C", All in one
"about 40 minutes" in the README too.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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
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.
Summary
Changes from a UX review of the whole window. Only text, icons, button order, window size and keyboard use changed. A run, how it is stopped, what is put back afterwards and every safety check are unchanged.
--open-optimize=N).run.logis.gao --reset.boot.logand the command line keep the engine's wording; the Dashboard log rewords it.Not changed, by choice: progress per step ("step 2 of 5"), screen-reader support, and editing the fan curve with the keyboard.
Before and after
All pictures are
PrintWindowcaptures of the--demowindow, run without administrator rights. Most are 1180×1000; the small-window ones are at the new minimum, 1000×700 (the old pictures were taken at 960×700).The pictures are kept on the branch
pr-assets/ux-review, so they never reachmain. That branch can be deleted once this is merged.Test plan
cmake --workflow --preset ci: builds with/W4 /WX,core_testsandapp_testspass.--demoand--demo=run(without administrator rights): went through the question about other programs, every run step, the result, Abort, "Don't start", the Dashboard, the Fan page and the About page, at 1180×1000 and at the 1000×700 minimum; Tab, Enter and Space were tried too.--open-optimize=N;🤖 Generated with Claude Code