-
Notifications
You must be signed in to change notification settings - Fork 144
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
Implement key binds to control virtual outputs #1287
Conversation
It is now possible to use keybinds to add and remove virtual outputs (also called headless backend in wlroots terminology).
@Consolatis thanks for your review. I've addressed your comments. Do I need to squash everything into a single commit? |
@kyak up to you. In general yes but we can also do that when merging. As this is a feature that I helped to design and we usually don't self-merge I'd prefer if @johanmalm could have a final look over this PR. From my POV this is ready to go after squashing. |
Happy in principle. Just a bunch of fairly trivial comments. |
@johanmalm thanks! I've addressed most of your comments. Kept it in separate to-be-squashed commits for better traceability. |
Great effort. Thanks! |
It is now possible to use keybinds to add and remove virtual outputs (also called headless backend in wlroots terminology).
Closes #1279
Thanks to @Consolatis for original PoC patch and guidance!