haxiomic / firefox-multi-touch-zoom

Smoothly zoom in and out with the multi-touch pinch gesture
MIT License
154 stars 16 forks source link

It affects Ctrl+wheel zoom #43

Open HubKing opened 5 years ago

HubKing commented 5 years ago

I need both (1)the default zoom that changes the page layout and (2)the multi-touch zoom that enlarges a portion without changing the layout. What I expected was Ctrl+wheel for (1) and Shift+wheel for (2).

If I disable this extension, Ctrl+wheel works for (1), Shift+wheel does nothing.

If I enable this extension, both Ctrl+wheel and Shift+wheel work for (2). This is not what I wanted. Is this a bug? FireFox 69.0.2 on Windows with a keyboard and mouse.

haxiomic commented 4 years ago

Unfortunately this is a quirk in the way two finger pinch is handle on windows firefox - the pinch gesture get mapped to scroll + ctrl, and there’s no way to distinguish between pinching and scroll + ctrl

Native pinch zoom is in dev now so this plugin will be retired soon. You could try enabling it to see if it works better. Here’s instructions: https://github.com/haxiomic/firefox-multi-touch-zoom/issues/42#issuecomment-537702519

HubKing commented 4 years ago

Thank you, sir, but my PC does not even have a multi-touch device, so in my case, I do not need to map Ctrl+wheel (or as you say 'scroll + ctrl') to the (2). Would you add an option not to listen to the "Ctrl+wheel" event to this add-on?

Unless FireFox's upcoming native pinch-to-zoom provides a way to invoke (2) without a trackpad or a touchscreen, I would still need this add-on. For example, Chrome already supports (2), but Chrome's (2) can only be invoked by a trackpad or a touchscreen, so for a desktop PC without a trackpad or a touchscreen, which should be the majority of the desktop PCs, this kind of extnsion would be useful.