nion-software / nionswift

Nion Swift is open source scientific image processing software integrating hardware control, data acquisition, visualization, processing, and analysis using Python. Nion Swift is easily extended using Python. It runs on Windows, Linux, and macOS.
http://nion.com/swift
GNU General Public License v3.0
44 stars 33 forks source link

Add zoom tool, zoom menu items #21

Open cmeyer opened 7 years ago

cmeyer commented 7 years ago

From Tracy: For zooming in an out, we use "+", but then you are always zooming about the center. If we use the track wheel on the mouse, we could zoom about the cursor, and then you could zoom in on a particular feature.

From Niklas: The best tracking devices, track balls often come without a scroll wheel, so making + and - move around the cursor would be better. There is a usability issue in Swift in that it doesn't actually know where the cursor is, but it should.

### Tasks
- [ ] https://github.com/nion-software/nionswift/issues/1137
- [ ] Add zoom menu items (zoom around mouse if cursor in panel, otherwise center)
- [ ] Add zoom by scroll wheel / track pad (zoom around mouse position)
Brow71189 commented 4 years ago

Maybe both ideas should be implemented? But I definitely think Tracy's idea should, because this is what the mouse wheel does in most applications. Right now it moves images up/down in Swift, which is not very useful. A nice side effect of zooming around the cursor is that you can use that to move the image around. Moving images is often a bit awkward in many programs, so using "zoom around the cursor" is a decent workaround.

Niklas' idea might be useful, too, but I 'm afraid that this can be confusing for users. Maybe we should just get trackballs with mouse wheels?

ndellby commented 4 years ago

I've looked, none of the good index finger trackballs come with scroll wheels, they only have the little minibuttons.

Niklas

On 5/26/2020 8:36 AM, Andreas Mittelberger wrote:

Maybe both ideas should be implemented? But I definitely think Tracy's idea should, because this is what the mouse wheel does in most applications. Right now it moves images up/down in Swift, which is not very useful. A nice side effect of zooming around the cursor is that you can use that to move the image around. Moving images is often a bit awkward in many programs, so using "zoom around the cursor" is a decent workaround.

Niklas' idea might be useful, too, but I 'm afraid that this can be confusing for users. Maybe we should just get trackballs with mouse wheels?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/nion-software/nionswift/issues/21#issuecomment-634102817, or unsubscribe https://github.com/notifications/unsubscribe-auth/ALBPXDZJ6BHEGPNWPZPTEL3RTPOZRANCNFSM4DXJ7LEA.

Brow71189 commented 4 years ago

When I looked I found a couple. For example https://www.amazon.com/dp/B01936N73I/ref=cm_sw_r_cp_apa_i_pT83EbD923DSJ

Or

https://www.amazon.com/dp/B002OOWB3O/ref=cm_sw_r_cp_apa_i_UU83EbK0H5M6V

Or

https://www.amazon.com/dp/B0050SPZMK/ref=cm_sw_r_cp_apa_i_HV83EbKDZC718

Or

https://www.amazon.com/dp/B07J3XZYGG/ref=cm_sw_r_cp_apa_i_.W83EbJTYQY61

Or

https://www.amazon.com/dp/B0735584RM/ref=cm_sw_r_cp_apa_i_IY83Eb66VPXCW

Actually I think having a scroll wheel would be nice also for general use of the user pc.

ndellby notifications@github.com schrieb am Di., 9. Juni 2020, 19:16:

I've looked, none of the good index finger trackballs come with scroll wheels, they only have the little minibuttons.

Niklas

On 5/26/2020 8:36 AM, Andreas Mittelberger wrote:

Maybe both ideas should be implemented? But I definitely think Tracy's idea should, because this is what the mouse wheel does in most applications. Right now it moves images up/down in Swift, which is not very useful. A nice side effect of zooming around the cursor is that you can use that to move the image around. Moving images is often a bit awkward in many programs, so using "zoom around the cursor" is a decent workaround.

Niklas' idea might be useful, too, but I 'm afraid that this can be confusing for users. Maybe we should just get trackballs with mouse wheels?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub < https://github.com/nion-software/nionswift/issues/21#issuecomment-634102817>,

or unsubscribe < https://github.com/notifications/unsubscribe-auth/ALBPXDZJ6BHEGPNWPZPTEL3RTPOZRANCNFSM4DXJ7LEA .

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/nion-software/nionswift/issues/21#issuecomment-641456611, or unsubscribe https://github.com/notifications/unsubscribe-auth/AD7SKK66ZJ4IL52AVWZVCP3RVZU5TANCNFSM4DXJ7LEA .

jamesrussell216 commented 1 month ago

PR for this currently is: https://github.com/nion-software/nionswift/pull/1073

However we're not currently happy with the two button concept so we're going to take another look at that first before moving this forward.

jamesrussell216 commented 1 week ago

After discussion on this issue we've got a proposed revised spec that @Tiomat85 is going to code too...