search

jcbastosportela / logioptspp

Smooth scrolling enabler for Logitech mice.
MIT License
50 stars 1 forks source link

Feature request: Support whitelisting of apps #10

Closed pouldam closed 6 months ago

pouldam commented 10 months ago

It would be nice if the app supported the option to use a whitelist instead of a blacklist.

I only have one additional application where I would like to add the smooth scrolling, thus a whitelist would make it a lot easier to configure.

One of the reasons why I do not want smooth scrolling enabled for all apps is that configuration of mouse buttons per app level goes down the drain as every app inherits the configuration for the Chrome browser.

MaxHasBeenUsed commented 6 months ago

You could try https://github.com/jcbastosportela/logioptspp/releases/tag/v0.0.1 for now since that's what's changed in d5b5f58e010788919a8df7debc6cec3de2e1a986

Personally I prefer whitelist mode as well. Would be best if there could be an option to choose from the two modes some day.

pouldam commented 6 months ago

Perfect! - that sounds exactly like what I am seeking. However, when I try to launch v0.0.1 of your app I get an error message stating that ucrtbased.dll was not found. Closing the dialog I get the same fault just with VCRUNTIME140_1D.dll, MSVCP140D.dll and VCRUNTIME140D.dll.

I guess this is all related to v0.0.1 being a debug build?

Would you suggest I just seek out the debug libraries on the internet or do you have a release version in your drawer somewhere?

Thanks in advance.

Oh - and soooo cool to finally have smooth scrolling in more apps. I don't know why Logitech miss out on that opportunity.

MaxHasBeenUsed commented 6 months ago

Sorry but you may have mistaken me for the owner of this repo?

I'm assuming you got the errors when trying to build from source code? If that's the case you could just simply download and unzip v0.0.1.zip found in release page. Then you want to edit the conf.ini to add whitelisted apps and choose a working mode (PRELOAD/INJECT). After saving the ini, simply run logioptionspprun.exe should be good.

Hope it helps!

pouldam commented 6 months ago

Ah, sorry - yeah I thought you were the author :-)

Anyway, I actually downloaded the v0.0.1 build from the release page and it was that one that gave me the errors (v0.0.2 runnning fine). I can see that the release notes for v.0.0.2 says that the build has been changed to a release build. Thus I assume that the v0.0.1 release is a debug build. That also matches the errors I am getting - the names of the missing DLLs all end on the letter 'd' which makes me assume that they are debug DLLs which my system currently lacks.

I guess the missing DLLs should be pretty easy to dig up from a reliable source (Microsoft), so I will just give that a shot.

Thanks for pointing me in the right direction!

jcbastosportela commented 6 months ago

@pouldam check this PR. I think it now does what you want.

https://github.com/jcbastosportela/logioptspp/pull/13

jcbastosportela commented 6 months ago

https://github.com/jcbastosportela/logioptspp/releases/tag/v0.0.3