Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

21 - Adding zoom in/out buttons to toolbar. #1

Merged
merged 2 commits into from
May 15, 2024
Merged

21 - Adding zoom in/out buttons to toolbar. #1

merged 2 commits into from
May 15, 2024

Conversation

Tiomat85
Copy link

@Tiomat85 Tiomat85 commented May 15, 2024

nion-software#21

Created ZoomMouseHandler, and an apply_fixed_zoom that takes a zoom target coordinate

Created ZoomMouseHandler, and an apply_fixed_zoom that takes a zoom target coordinate
@Tiomat85 Tiomat85 requested review from lisham2000 and KRLango May 15, 2024 09:49
nion/swift/ImageCanvasItem.py Show resolved Hide resolved
nion/swift/resources/mag_glass_in.png Outdated Show resolved Hide resolved
nion/swift/test/ImageCanvasItem_test.py Show resolved Hide resolved
@Tiomat85 Tiomat85 merged commit cd6050f into master May 15, 2024
@jamesrussell216 jamesrussell216 changed the title Adding zoom in/out buttons to toolbar. 21 - Adding zoom in/out buttons to toolbar. May 20, 2024
@Ion-e
Copy link

Ion-e commented May 21, 2024

The Zoom button does not consistently work when having used the "Zoom to fit/Zoom image to 1/2 pixels" tools.

After using these tools and using the Zoom buttons, there will be unexpected "jumps".

@Ion-e
Copy link

Ion-e commented May 21, 2024

The Zoom buttons are greyed out, inconsistently with the rest of the "handling" tools in the toolbar.

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.

3 participants