Fruchtzwerg94 / PlantUmlViewer

A Notepad++ plugin to generate, view and export PlantUML diagrams
MIT License
82 stars 9 forks source link

Remove zoom on scroll #7

Closed thijsvanderkraan closed 2 years ago

thijsvanderkraan commented 2 years ago

I often accidentally scroll while the mouse is in the Plant preview area. Scrolling is directly linked to zooming, and zooming in or out does not improve the readability

would prefer scrolling behaves in the same way as notepad itself, scroll = vertical scroll ALT-scroll = horizontal scroll SHIFT-scroll = zoom

Fruchtzwerg94 commented 2 years ago

Hi @thijsvanderkraan ,

thanks for the feedback. Indeed the zoom and scrolling behaviour is not really the common way. The specific proposal you've made looks not very common and also not implemented in Notepad++ for me. I think the following should be the most intuitive / common solution (also in Notepad++ by default):

This is exactly what I've implemented in the latest commit since I like the idea to align to the "standard" here. Please have a look at the latest build, try it out and give some feedback on how it works for you: Build for c42a80b

Cheers, Philipp

thijsvanderkraan commented 2 years ago

Dear Philip,

You are absolutely right, I didn't write it down correctly. With scroll I meant the mouse wheel scroll button and I got the shift and alt mixed up. I tested c42a80b and it works like a charm, thank you!

Best regards, Thijs