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

swaybar protocol: handle button 4 and 5 #1368

Merged
merged 1 commit into from
Sep 23, 2017

Conversation

johnae
Copy link
Contributor

@johnae johnae commented Sep 23, 2017

i3 bar handles scroll events as button 4 and 5. An example use case would be volume up/down.
This probably has some effect on the "scrolling between workspaces" feature - now one probably has to put the pointer over the workspace buttons for it to work.

@ddevault
Copy link
Contributor

Thanks for the patch!

now one probably has to put the pointer over the workspace buttons for it to work.

This isn't going to work - can you fix this?

@johnae
Copy link
Contributor Author

johnae commented Sep 23, 2017

Yeah, I'm very unfamiliar with the code base atm - just started looking at it today. Should be quite easy to fix I believe.

i3 bar handles scroll events as button 4 and 5. An example use
case would be volume up/down.
@johnae
Copy link
Contributor Author

johnae commented Sep 23, 2017

@SirCmpwn There. Much better overall i.m.o and workspace switching should work as before (unless over the status bar items ofc).

@ddevault
Copy link
Contributor

Yes, that looks much better, thanks!

@ddevault ddevault merged commit a4e3b4f into swaywm:master Sep 23, 2017
@ddevault ddevault mentioned this pull request Oct 7, 2017
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants