Fix inverted scrollwheel direction in keybinds - #14749
Merged
Merged
Conversation
|
Hello and thank you for making a PR to Hyprland! Please check the PR Guidelines and make sure your PR follows them. If your code can be tested, please always add tests. See more here. beep boop, I'm just a bot. A real human will review your PR soon. |
10 tasks
vaxerski
approved these changes
May 22, 2026
vaxerski
left a comment
Member
There was a problem hiding this comment.
one of the moments of all time
|
Probably worth mentioning in breaking changes tracker since this will break literally every config |
Member
|
well no it will fix every cofig that used mosue_ to bind anything, I doubt many do tbh |
ItsOhen
pushed a commit
to ItsOhen/Hyprland
that referenced
this pull request
Jun 1, 2026
Kitty-Hivens
added a commit
to Kitty-Hivens/YukiUI
that referenced
this pull request
Jul 20, 2026
Hyprland 0.56.0 corrects a long-standing inversion where mouse_up/mouse_down in keybinds fired on the opposite physical wheel direction (hyprwm/Hyprland#14749). That flipped Super+scroll workspace focus and Super+Shift/Alt+scroll window-move relative to the wheel. Swap the bind prefixes so scroll-down again lands on the next workspace.
|
it broke everything. lost my job, family |
lwestlund
added a commit
to lwestlund/dotfiles
that referenced
this pull request
Jul 23, 2026
RayJameson
added a commit
to RayJameson/dotfiles
that referenced
this pull request
Jul 24, 2026
alonso-herreros
added a commit
to alonso-herreros/dotfiles
that referenced
this pull request
Aug 3, 2026
Since hyprwm/Hyprland#14749 mouse_up and mouse_down have been swapped.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Describe your PR, what does it fix/add?
The above binds are an easy way to show the issue: when scrolling down, you'll see the "Mouse up" notification and vice versa. The if statement needs to be inverted.
Checking with
wev, scrolling down yields:so a positive delta.
Is there anything you want to mention? (unchecked code, possible bugs, found problems, breaking compatibility, etc.)
This has been broken for 4 years (from its initial implementation in #265) so this might break some people's configs.
Is it ready for merging, or does it need work?
Ready. I didn't test it though but this seems pretty trivial.