Skip to content
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

Add support for 8 and 9 mouse buttons #843

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

githaff
Copy link

@githaff githaff commented Sep 6, 2018

Some mouses have additional buttons which can be conveniently used for window
move/resize operations. Buttons 4-7 are skipped since they represent scroll
events.

Dmitry Lavnikevich added 2 commits September 6, 2018 20:56
Some mouses have additional buttons which can be conveniently used for window
move/resize operations. Buttons 4-7 are skipped since they represent scroll
events.
Unlike buttons 1-3 these two can be used solely for window resize/movement
actions and so do not require modifier.
@githaff
Copy link
Author

githaff commented Sep 6, 2018

The end game is to use mouse buttons 8 and 9 to resize and move windows without the need of touching a keyboard. So unlike buttons 1-3, these should work without the modifier.
In my repo I have disabled need for a modifier for pointer grab on 8-9 buttons. But that seems hacky and contradicting existing documentation. What would be the best approach to implement it?

Dmitry Lavnikevich added 2 commits May 29, 2019 23:55
This prevents pointer being centered on a node when the option is configured.
It helps in case when xbanish only enables pointer following focus when the pointer
is invisible. Which is needed to prevent 'caging' pointer on some modal dialog
windows.
Dmitry Lavnikevich and others added 3 commits November 10, 2019 12:42
Split node vertically only when width exceed height of node hvsplit_ratio times.
Since gcc 10 compiler defaults to -fno-common. In bspwm it is fixed in master.
Set flag explicitly to avoid pulling in changes from master.
@jvvv
Copy link
Contributor

jvvv commented Oct 6, 2024

This pr does much more than what it says. Only 2 of the 7 commits deal with the topic. The others aren't even related.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants