search

surge-synthesizer / shortcircuit-xt

Will be a sampler when its done!
GNU General Public License v3.0
257 stars 31 forks source link

Mousewheel direction #1263

Open Andreya-Autumn opened 2 months ago

Andreya-Autumn commented 2 months ago

It's going the wrong way for me now (on MBP trackpad). Maybe add a reverse switch in a menu somewhere?

baconpaul commented 2 months ago

I picked it so it is identical to logic and audacity

do you have wheel direction drags reversed in your os or some such?

mkruselj commented 2 months ago

We should absolutely offer scrolling direction option (both vertical and horizontal scrolling).

baconpaul commented 2 months ago

macOS has it as an os setting - shouldn’t we just follow that in Mac?

baconpaul commented 2 months ago

Settings / mouse / natural scrolling basically

baconpaul commented 2 months ago

HScroll

yeah so i compared both h and vscroll with the setting on and off at the os level and sc does the same thing as logic. On mac at least there's not a bug and i don't think there's a need for a setting.

mkruselj commented 2 months ago

Well ok then maybe no setting for macOS, but for Win/Lin for sure there would have to be IMO (touchpad vs mousewheel users have different expectations).

Andreya-Autumn commented 2 months ago

Ok back at the studio and tested with my mouse now. The mousewheel does nothing at all! Can't get it to zoom or scroll regardless of key combinations.

Andreya-Autumn commented 2 months ago

Oh, not quite true actually. It does vertical scroll if I first zoom in vertically with the mousepad.

baconpaul commented 2 months ago

https://github.com/surge-synthesizer/sst-jucegui/blob/e321b9b6d6b298d9d2aed9f459a4a2ae8a540b8a/include/sst/jucegui/components/ZoomContainer.h#L107 well there’s the code which is handling it

when I do a wheel like gesture in my track pad with shift or shift alt held down it does work

I would start with a a print there tbh