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

[wayland] Add environment variable to configure cursor size #4681

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

plusgut
Copy link

@plusgut plusgut commented Feb 4, 2024

At X11 the way to configure the cursor_size is done by configuring XCURSOR_SIZE https://man.archlinux.org/man/Xcursor.3#XCURSOR_SIZE

There sadly is no standardized way to do that in wayland, the weston-example compositor also does it by XCURSOR_SIZE https://man.archlinux.org/man/weston.1.en#XCURSOR_SIZE
https://gitlab.freedesktop.org/wayland/weston/-/blob/master/clients/window.c#L1338

I added the same approach to qtile with this pr

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for opening your first PR into Qtile!
If you have not heard from us in a while, please feel free to ping one of the devs or anyone who has commented on the PR, as sometimes things fall through the cracks.
You can also join the chat room for real-time discussion, see the community links.
For details on what PRs might need to include before we will merge them please see the docs.

@jwijenbergh
Copy link
Contributor

Sure, makes sense. But what about making it a config option instead?

@plusgut
Copy link
Author

plusgut commented Feb 5, 2024

@jwijenbergh sure I can do that. Should it be a wayland-specific option? How to tell the X11 backened the cursor size is not obvious to me, and I would need a couple pointers.

@jwijenbergh
Copy link
Contributor

jwijenbergh commented Mar 24, 2024

Yes I think prefixing it with wl_ in the config is a good idea

@jwijenbergh
Copy link
Contributor

This has been added in the wlr 0.17 pr with a wl_xcursor_size variable in the config

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.

None yet

2 participants