pixop / video-compare

Split screen video comparison tool using FFmpeg and SDL2
GNU General Public License v2.0
889 stars 41 forks source link

request: zoom with the mouse scroll wheel #54

Closed RealSting closed 6 months ago

RealSting commented 7 months ago

Hi! thanks for this beautiful application! Is it possible to implement "zoom with the mouse scroll wheel". More or less like on YUView application. thanks :)

jonfryd commented 7 months ago

Hi @RealSting,

Glad to hear you find it useful!

Just to be sure: Would you like the ability to use the mouse wheel to change the size of the source area for the zoom-in feature? Currently, this area is locked at 64x64 pixels which might not be appropriate for all use cases.

I think it's a very good idea and should be easy to add.

RealSting commented 7 months ago

Hi Jon, thanks for quick feedback. Try YUViev please:

jonfryd commented 7 months ago

OK, I installed YUView and played with it. Cool feature, I'll think about something similar that makes sense for video-compare.

jonfryd commented 7 months ago

@RealSting: I've implemented the feature as requested. Check out the global_zoom branch if you are able to compile the software yourself, or you can wait for the next release. I'm aiming for one last release this year, later in December.

RealSting commented 6 months ago

@RealSting: I've implemented the feature as requested. Check out the global_zoom branch if you are able to compile the software yourself, or you can wait for the next release. I'm aiming for one last release this year, later in December.

Thans Jon for feedback, anyway I see only a branch: master

jonfryd commented 6 months ago

Thans Jon for feedback, anyway I see only a branch: master

Yeah, I merged both feature branches to master this morning prior to releasing the new version.

Thanks for suggesting the feature, I think it's a very useful addition to the tool. I'm closing the issue.

RealSting commented 6 months ago

This new feature is super! Thanks so much for quick feedback! Can you add also "click and hold down left/right mouse button to pan" ? :)

jonfryd commented 6 months ago

This new feature is super! Thanks so much for quick feedback!

Cool, good to hear!

Can you add also "click and hold down left/right mouse button to pan" ? :)

Yes, next year. Please create a separate issue :)