-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Can you add a config to disable all ALT key mechanics? #49
Comments
The modifier key (left ALT by default) can be rebound in the controls menu. I suspect the cause of the issue would be that Multi-Hot bar cancels the scroll event so it does not propagate to the vanilla handler. Unfortunately, it also means the event will not be propagated to other mods. Would you list the other mods that have conflicts? |
Right now, I can only think of Draconic Evolution's "Enhanced Charm of Dislocation". I'm pretty sure I removed the other 2 mods from my pack for now, so I'll get back to you on that. |
Had time for this today, but I can't seem to remember the 2 removed mods. Maybe you could just implement a little compat for the Charm? Don't know how hard that would be. |
I looked into it and the issue is with the scrolling, not the keybinds. Draconic Evolution specifically does not cancel the scroll event, instead, it changes the current item. The better solution would be to cancel the event so that vanilla/Multi-Hotbar does not handle it. I have submitted a PR Draconic-Inc/Draconic-Evolution#1061. In the meantime, there is a hotkey that will toggle between vanilla and Multi-Hotbar. I will (hopefully soon) add a config option that disables scrolling whilst holding down a pre-defined button for mods that do not cancel the scroll event. |
Some mods do not cancel the scroll event. The ignore scroll key can be bound to any key without worry of conflicts and the scrolling handler will ignore scroll events while this button is down. Fix #49
I've got about 3 mods that make use of the mouse wheel with the ALT key held. Sadly, none of them work with this, because your API takes away that option.
The text was updated successfully, but these errors were encountered: