phenax / bsp-layout

Manage layouts in bspwm (tall and wide)
MIT License
373 stars 30 forks source link

Replace cycle with previous and next in layout switcher #41

Closed totoroot closed 2 years ago

totoroot commented 2 years ago

This is a proposed fix for the issue raised in #40.

It was simple enough and IMO this would be useful indeed, as it would enable me to use it in a polybar module for bsp-layouts I'm currently working on. That way I could use the two directions of the mouse wheel to go back and forth through the list of layouts when hovering over the polybar module.

Let me know if I should combine the functions into one, as I wasn't sure what the more sensible approach would be.

totoroot commented 2 years ago

@hamensman Would you care to test the functionality and tell me if it's how you've imagined it, since you proposed the feature

phenax commented 2 years ago

Thanks for the pr!