Skip to content

Show current preset/config name in the top bar#1592

Open
CrunchyBanana1 wants to merge 1 commit into
Nerogar:masterfrom
CrunchyBanana1:preset-name-topbar
Open

Show current preset/config name in the top bar#1592
CrunchyBanana1 wants to merge 1 commit into
Nerogar:masterfrom
CrunchyBanana1:preset-name-topbar

Conversation

@CrunchyBanana1

@CrunchyBanana1 CrunchyBanana1 commented Jul 5, 2026

Copy link
Copy Markdown
Contributor

Adds a label in the PySide6 top bar showing the loaded preset or config name, with a " (*)" marker when the config differs from what was loaded or last saved. The name is restored across restarts via a small per-machine state file (gitignored, like #.json).

The Ctk top bar is unaffected: the display hooks are no-ops there.

Summary

Added the currently chosen Preset to the top bar in the preset dropdown.

grafik

AI assistance

  • AI-created: This is completely done by claude

@dxqb

dxqb commented Jul 11, 2026

Copy link
Copy Markdown
Collaborator

I like the idea of showing the selected preset again, but it's too much code to maintain that even saves a new file just to show the (*) that you have changed something. please remove that

@dxqb
dxqb changed the base branch from merge to master July 12, 2026 09:05
The "Load Preset" button now shows the loaded preset or config name, and
falls back to "Load Preset" when nothing is loaded. The name is set when a
preset or config is loaded or saved.

The Ctk top bar is unaffected: the display hooks are no-ops there.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@CrunchyBanana1
CrunchyBanana1 force-pushed the preset-name-topbar branch 2 times, most recently from 35304da to 50c144d Compare July 12, 2026 11:42
@CrunchyBanana1

Copy link
Copy Markdown
Contributor Author

I'm with you there. Felt aready a bit over the top for such a small thing.
I changed it to just the display of the current preset.

@dxqb

dxqb commented Jul 18, 2026

Copy link
Copy Markdown
Collaborator

looks good. one nitpik: it only works on Qt, not on Ctk.
at some point, we'll probably diverge (implement new features only on Qt) or even discard Ctk.

but it shouldn't be on a small feature I think. do you want to add it? should be easy with Claude
test with run-cmd train_ui_ctk

@CrunchyBanana1

Copy link
Copy Markdown
Contributor Author

Oh wow I could have sworn it already worked like that on Ctk, that's why I didn't even bat an eye.
Will take a look at it tomorrow of course :)

@dxqb dxqb added the preview merged in the preview branch label Jul 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

preview merged in the preview branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants