Skip to content

Tooltip readability fixes + clearer Batch Analysis crop warning#425

Merged
marcinz606 merged 4 commits into
marcinz606:mainfrom
linkmodo:feature/tooltip-shortcut-and-batch-warning
Jul 9, 2026
Merged

Tooltip readability fixes + clearer Batch Analysis crop warning#425
marcinz606 merged 4 commits into
marcinz606:mainfrom
linkmodo:feature/tooltip-shortcut-and-batch-warning

Conversation

@linkmodo

@linkmodo linkmodo commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Four small UX fixes across tooltips and the Batch Analysis dialog.

Tooltip keyboard shortcuts

  • Shortcut chips used to butt directly against the tooltip text. They now sit on their own line below the text, still inside the tooltip box.
  • When an action has two bindings they're separated with "&" (previously they ran together).
  • The "&" separator inherits the tooltip text colour so it's legible (the key chips stay greyed).
  • The shortcut row is right-aligned to balance the box against the left-aligned label.

Crosstalk "Matrix" tooltip wrapping

The Crosstalk Matrix tooltip was a long plain-text string. Qt only word-wraps tooltips it treats as rich text, so it rendered as a single line that ran across the whole screen. It's now wrapped in a fixed-width table so Qt word-wraps it to the controls-panel width.

Batch Analysis warning

The Batch Analysis confirmation dialog now:

  • Leads with the crop-status count (e.g. "0 of 12 files are cropped").
  • Strongly recommends cropping every image in the session first, explaining that uncropped frames leave the Analysis Buffer to trim only a small centered margin — so sprocket holes and empty space outside the frame get pulled into the luma/colour average and skew the result for the whole roll.
  • Retains the note that the Analysis Buffer and Luma Range Clip from the current frame are applied to every file before averaging.

Tests

tests/test_shortcut_registry.py extended for the new tooltip layout (own line, "&" separator, right-alignment, plain-text passthrough). Lint clean.

linkmodo added 4 commits July 8, 2026 18:53
Keyboard shortcuts appended to tooltips previously ran directly against
the tooltip text with no separation. Put the shortcut chip(s) on a new
line below the tooltip text (still inside the tooltip box) and, when an
action has two bindings, separate them with "and".
Rewrite the Batch Analysis confirmation dialog so the crop status count
comes first, followed by a strong recommendation to crop every image in
the session before running. Explain that uncropped frames leave the
Analysis Buffer to trim only a small centered margin, so sprocket holes
and empty space outside the frame get pulled into the luma/colour
average and skew the result for the whole roll. Keep the note that the
Analysis Buffer and Luma Range Clip from the current frame are applied
to every file.
Refine the tooltip shortcut row: separate two bindings with "&" instead
of "and", let that separator inherit the tooltip text colour (the chips
stay greyed) so it is legible, and right-align the shortcut row within
the tooltip box to balance it against the left-aligned text.
The Crosstalk "Matrix" tooltip was a long plain-text string. Qt only
word-wraps tooltips it treats as rich text, so this one rendered as a
single line that ran across the whole screen. Wrap it in a fixed-width
table so Qt word-wraps it to the controls panel width.
@marcinz606 marcinz606 merged commit 7e20097 into marcinz606:main Jul 9, 2026
1 check passed
marcinz606 added a commit that referenced this pull request Jul 9, 2026
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.

2 participants