v1.2.3: Reader UX polish — Fit Page, Ctrl+Wheel zoom, clearer zoom controls#62
Merged
Conversation
…ntrols - **Fit Page on open:** _effective_zoom now considers both width and height constraints so every PDF opens with the first page fitting cleanly inside the document viewport. Applies to normal open, recent files, session restore, and new tabs. - **Ctrl+Mouse Wheel zoom:** Added wheelEvent handler on the scroll area viewport. Ctrl+Wheel Up zooms in, Ctrl+Wheel Down zooms out. Wheel scrolling is suppressed while Ctrl is held. - **Clearer zoom buttons:** Replaced unicode minus (\u2212) with a standard '-' label. Buttons are now - / + / Fit with slightly wider fixed widths (32/32/44) and improved tooltips mentioning mouse wheel. - **Toolbar spacing:** Increased spacing between page controls, zoom group, and copy/search from 4px → 6px for a cleaner, less cramped layout in both light and dark themes. - **Version bump:** __version__ → 1.2.3, MSIX → 1.2.3.0. - **Tests:** Added regression coverage for zoom constants, button labels, Ctrl+Wheel wiring, effective-zoom logic, and toolbar tooltips. - **Changelog:** Added v1.2.3 entry. Co-Authored-By: Claude <noreply@anthropic.com>
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
1 participant
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
Focused reader UX polish release for OpenReader v1.2.3.
Changes
Default Fit Page on open — All PDF open paths (normal open, recent files, session restore, new tab) now start with the page fitting cleanly inside the viewport. Uses both width and height constraints for true Fit Page behavior.
Ctrl+Mouse Wheel zoom — Scroll up with Ctrl held to zoom in, scroll down to zoom out. Page scrolling is suppressed while Ctrl is held.
Clearer zoom controls — Replaced the unicode minus sign button with a standard
-label. Buttons now read-/+/Fitwith improved tooltips.Toolbar spacing fix — Increased spacing between page controls, zoom group, and copy/search controls from 4px to 6px for a cleaner layout in both light and dark themes.
Version bump — App →
1.2.3, MSIX →1.2.3.0.Tests added — Regression coverage for zoom constants, button labels, Ctrl+Wheel wiring, fit-to-page logic, and toolbar tooltips (37/38 pass; the 1 failure is a pre-existing README text change).
Changelog — Added v1.2.3 entry with full release notes.
Files changed
main.py_effective_zoomCHANGELOG.mdpackaging/msix/AppxManifest.xmlpackaging/msix/AppInstaller.xmltests/test_reliability.pyAcceptance criteria
-,+, andFitStore release notes
🤖 Generated with Claude Code