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
Fix master layout window focus and scroll #5074
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm but I'd also like someone to test this from the affected issues
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<3
Took a look at the issues, I realize there's still |
* fix master switch window scrolling * fix some more dispatchers and remove some duplicate code * refactor and remove duplicate code * fix focusmonitor: hyprwm#5006 (comment) * change check
Describe your PR, what does it fix/add?
Fixes #4127
Fixes #5006
Removed some duplicate code
Is there anything you want to mention? (unchecked code, possible bugs, found problems, breaking compatibility, etc.)
This shouldn't change any bahavior other than fixing the issue.
In the future it's probably better to make
switchToWindow
use common code withCKeybindManager::switchToWindow
Is it ready for merging, or does it need work?
Ready