gui: status bar operable split, no busy overlay in Sample alignment #145

Merged

duan_j merged 4 commits from feat/gui-dynamic-signals-and-sample-tools into main 2026-08-18 09:27:42 +02:00

Conversation 1 Commits 4 Files Changed 8

+142 -21

duan_j commented 2026-08-18 09:20:33 +02:00

Owner

Copy Link

Copy Source

Three changes on top of main:

• fix: no BEAMLINE BUSY overlay during Sample alignment. Same exception as the motion watch: busy in Sample alignment is the alignment itself, and the sample motion is watched in these very views - the curtain hid exactly what the user needed to see. Gated at both style call sites (sample camera and the Axis views); the session-driven 'In viewing mode' badge is unaffected.
• feat: status bar splits passive left / operable right. Operable controls (Fast Shutter, State, p-group, Session) group right via addPermanentWidget and share one hover affordance (hand cursor + underline) now owned by ClickableLabel; ValueLabel opts out for the passive readouts, and the unwired busy label becomes a plain QLabel. Cryo sits first in the operable group but stays passive; placement reserves its spot for the planned cryo operation.
• chore: update banner png so the letters have gaps (215 KB -> 31 KB).

Generated with Claude Code

Three changes on top of main: - fix: no BEAMLINE BUSY overlay during Sample alignment. Same exception as the motion watch: busy in Sample alignment is the alignment itself, and the sample motion is watched in these very views - the curtain hid exactly what the user needed to see. Gated at both style call sites (sample camera and the Axis views); the session-driven 'In viewing mode' badge is unaffected. - feat: status bar splits passive left / operable right. Operable controls (Fast Shutter, State, p-group, Session) group right via addPermanentWidget and share one hover affordance (hand cursor + underline) now owned by ClickableLabel; ValueLabel opts out for the passive readouts, and the unwired busy label becomes a plain QLabel. Cryo sits first in the operable group but stays passive; placement reserves its spot for the planned cryo operation. - chore: update banner png so the letters have gaps (215 KB -> 31 KB). Generated with [Claude Code](https://claude.com/claude-code)

gitea-actions bot commented 2026-08-18 09:24:29 +02:00

Copy Link

Copy Source

Coverage report (automated)

Total line + branch coverage: 49%
Diff coverage vs main: 100% (minimum 80%)

Full report: coverage artifact on this run.

Coverage report (automated) Total line + branch coverage: 49% Diff coverage vs main: 100% (minimum 80%) Full report: coverage artifact on this run.

duan_j added 4 commits 2026-08-18 09:25:16 +02:00

fix: no BEAMLINE BUSY overlay during Sample alignment ... 16b61b886b

Same exception as the motion watch: busy in Sample alignment is the
alignment itself, and the sample motion is watched in these very
views - the curtain hid exactly what the user needed to see. Gated
at both style call sites (sample camera and the Axis views); the
session-driven 'In viewing mode' badge is unaffected.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

feat: status bar splits passive left / operable right ... 7a01a866ef

Operable controls (Fast Shutter, State, p-group, Session) group right
via addPermanentWidget and share one hover affordance - hand cursor +
underline - now owned by ClickableLabel; ValueLabel opts out for the
passive readouts, and the unwired busy label becomes a plain QLabel.
Cryo sits first in the operable group but stays passive: a cryo
operation is planned (TODO in status_bar.py), the placement just
reserves its spot.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

chore: update png so they have gaps 0f12b688fe

style: collapse multiline _status calls for ruff format ... 167d78db68

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

duan_j force-pushed feat/gui-dynamic-signals-and-sample-tools from f69da09038 to 167d78db68 2026-08-18 09:25:16 +02:00 Compare

duan_j merged commit 167d78db68 into main 2026-08-18 09:27:42 +02:00

duan_j deleted branch feat/gui-dynamic-signals-and-sample-tools 2026-08-18 09:27:42 +02:00

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

bug

Something is not working

duplicate

This issue or pull request already exists

enhancement

New feature

help wanted

Need some help

invalid

Something is wrong

question

More information is needed

wontfix

This won't be fixed

No labels

Milestone

No items

No Milestone

Projects

Clear projects

No projects

Assignees

Clear assignees

buntschu_d (buntschu_d) duan_j (duan_j) glettig_w (glettig_w) gobbo_a (gobbo_a) gotthard_g (gotthard_g) leonarski_f (leonarski_f) mx-gitea olieric (olieric) perl_d (perl_d) takaba_k (takaba_k) wang_m (wang_m)

No Assignees

2 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: mx/AareDAQ#145