-
Notifications
You must be signed in to change notification settings - Fork 90
Wayland fixes and cleanup #120
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
base: master
Are you sure you want to change the base?
Conversation
|
Let me know when you finish |
- Fix window not opening - Use minimum protocol versions - Fix compiler warnings - Do keyboard input properly using libxkbcommon (keymaps are respected) - Remove unnecessary code - Respect cursor theme - Support resizing and closing - Support scaling - Fix other issues stemming from logical errors/protocol violations
|
@Darky-Lucera I think I've done all I can. I have 2 questions though:
|
|
Hello! Excuse me for the long delay.
Best regards! |
Sounds like Wayland itself needs more work before its ready for prime-time, that sounds ridiculous. |
|
Hello! |
|
I didn't manage to make the viewport stuff work, but we could still merge this as is. Would you like me to revert anything? I think I unnecessarily changed some stuff that's only semantic (e.g., |
|
Hello! I've been working to get the MiniFB examples to compile and run on Wayland, which required some code changes. Since these might be relevant to your current work, I'd appreciate it if you could review the modifications. I apologize for any inconvenience. |
|
That... was the point of this PR as well. |
|
Could you please test the current version and apply you changes if needed? |
|
Are there some changes on this? |
The current implementation is a good basis, but off in a few places, which I attempted to fix in this PR.
Tell me if I should explain/revert anything.
Draft because I might still attempt some of the remaining problems. They are: