Skip to content

STM Background dialog: buttons above preview + auto-refresh on model change#16

Merged
jacobson30-bot merged 1 commit into
mainfrom
stm-background-dialog-ux
Jun 5, 2026
Merged

STM Background dialog: buttons above preview + auto-refresh on model change#16
jacobson30-bot merged 1 commit into
mainfrom
stm-background-dialog-ux

Conversation

@jacobson30-bot
Copy link
Copy Markdown
Contributor

Two UX tweaks to the STM Background dialog.

  • Buttons above the preview. The Preview background / Preview corrected image / Apply / Close row moves from the bottom of the dialog to directly under the controls, above the large image + plot panes, so the actions stay reachable without tracking down past the preview.
  • Auto-refresh on model change. Switching the background model now re-runs the current preview automatically (when a preview is already showing). In-model parameter tweaks still use the manual Preview buttons, as before.

Layout + small behavior change only; ruff clean, full suite green (1971 passed, 207 skipped). GUI rendering not executed in the sandbox (no Qt platform plugin) — worth a visual confirm on launch.

🤖 Generated with Claude Code

…del change

- Move the Preview background / Preview corrected image / Apply / Close row
  from the bottom of the dialog to directly under the controls, above the large
  image + plot preview panes, so the actions stay reachable without tracking to
  the bottom of the frame.
- Auto-refresh the preview when the background model is switched (re-runs the
  current preview mode if one is already showing). In-model parameter tweaks
  still use the manual Preview buttons.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@jacobson30-bot jacobson30-bot merged commit 482ce33 into main Jun 5, 2026
2 of 3 checks passed
@jacobson30-bot jacobson30-bot deleted the stm-background-dialog-ux branch June 5, 2026 15:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant