Streamline bottom toolbar; stabilize compare/flat-peek; tooltips#591
Merged
Conversation
- Remove the zoom slider from the toolbar (users zoom on the canvas directly); keep the read-only zoom % readout. - Move the GPU/CPU toggle off the editing row into the "More Actions" overflow menu as a checkable item next to Toggle HQ Preview, with its backend/status surfaced via the action tooltip. - Make Before/After and Peak Flat mutually exclusive both ways: enabling Before/After now clears an active Peak Flat so its toggle can't linger lit while the compare baseline is what's on screen. - Keep an active Before/After or Peak Flat across rotate/flip: geometry ops re-render within the current comparison view (new AppController.rerender_active_view) instead of dropping back to the plain edit. Only leaving the canvas exits the view.
- Zoom % readout now matches the 32px button height and centers on both axes so it sits on the icon line instead of floating; bumped to the base font at weight 600 for legibility. - Give every More Actions item a tooltip: the twinned actions reuse their toolbar tooltip (with the same shortcut badge), and the overflow-only items get concise descriptions. Menu already had setToolTipsVisible(True).
This was referenced Jul 22, 2026
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
2 participants
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
Bottom-toolbar cleanup plus two behavior fixes for the Before/After and Peek Flat comparison views.
Toolbar layout
Comparison-view behavior
AppController.rerender_active_view) using the rotated/flipped baseline or flat-master config, instead of dropping back to the plain edit. Only leaving the canvas exits the view.Test plan
TestCompareFlatPeekInteractionintests/test_controller.pycovers the mutual-exclusion and the threererender_active_viewbranches; proven non-vacuous by sabotagingrerender_active_viewand confirming failures.tests/test_canvas_toolbar.pyupdated for the relocated GPU control.MainWindow: slider gone / label styled, GPU toggle in the menu and wired togpu_enabled, Before/After clears Peek Flat, rotate/flip keep the active view, and every More Actions item carries a tooltip.ruff check+ruff formatclean. Full run of affected test files passes (except the pre-existingtest_effective_input_iccWindows path-separator failure, unrelated).