search

cyber-sushi / makima

Linux daemon to remap and create macros for keyboards, mice and controllers
GNU General Public License v3.0
138 stars 1 forks source link

Some mouse remaps no longer work since v0.9.0 #14

Closed pintassilgo closed 3 months ago

pintassilgo commented 3 months ago

This is my config file for my mouse:

[remap]
BTN_EXTRA-SCROLL_WHEEL_UP = ["KEY_VOLUMEUP"]
BTN_EXTRA-SCROLL_WHEEL_DOWN = ["KEY_VOLUMEDOWN"]
BTN_SIDE-SCROLL_WHEEL_UP = ["KEY_LEFTCTRL", "KEY_LEFTSHIFT", "KEY_TAB"]
BTN_SIDE-SCROLL_WHEEL_DOWN = ["KEY_LEFTCTRL", "KEY_TAB"]

The volume thing has always worked and it's still fine. But the last two lines to switch tabs using mouse wheel aren't working since v0.9.0.

cyber-sushi commented 3 months ago

Thank you for your report!

I just pushed v0.9.2, let me know if it fixes the issue for you

pintassilgo commented 3 months ago

Confirmed, thank you!

pintassilgo commented 3 months ago

By the way, just saying, some recent change appears to have fixed something I thought to be impossible.

The volume remap in first lines changes volume by 5% and triggers a Pasma widget displaying current value on the screen. But it was always throttled down (not sure if I'm using the right wording) so that turning mouse wheel fast didn't work properly. Example: if I turn wheel by 5 'ticks' in a single finger movement, only one was recognized to change volume, so I had volume changed by 5% and not by 25% as expected. It was this way for every remap app I tried, so I thought it to be a Plasma issue. I had to turn wheel slowly to be able to do big volume change.

But... somehow, now it's working as expected so I can finally change volume by a lot quickly. Thanks again.

cyber-sushi commented 3 months ago

In v0.9.0 i made some optimizations to reduce mutex locking and memory access, plus an optimization specific to wheel events (which is what initially broke the scroll wheel as mentioned in this issue), so that's probably why the events are being flushed faster now