-
-
Notifications
You must be signed in to change notification settings - Fork 161
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
Use of hyprcursor for hyprland #478
Comments
Thanks for pointing this out!
It was just not added yet. |
From the page you linked, I get the impression that Hyprcursor is simply an alternative file format for cursor themes. Are there any benefits to supporting this compared to using the traditional Xcursor format? According to the page, Hyprland supports both and will fall back to the traditional format if Hyprcursor is not configured. |
You can read https://github.com/hyprwm/hyprcursor for more information, but as far as I know, its overall better than xcursor |
The main downside is that Xcursor will still be required for other compositors / apps which don't support Hyprcursor, so we have to support both formats as well as converting between them if possible |
IMHO, Vaxry's high-quality projects might eventually become the standard.
Users not switching fully to Vaxry's ecosystem would face system bloat from duplicating functionalities. Would it be a good idea to integrate Hyprcursor into the Stylix Hyprland module? |
I support adding support for hyprcursors, but I am not sure if there are any hyprcursor themes in nixpkgs at the moment to test stylix support with. I use Someone could implement an overridable parameter for the Maybe somerthing like this will be approved by package maintainers: cursor.package = pkgs.bibata-cursors; # cursorFormat is set to xcursor by default for backwards compatibility
cursor.package = pkgs.bibata-cursors.override { cursorFormat = "xcursor"; };
cursor.package = pkgs.bibata-cursors.override { cursorFormat = "hyprcursor"; }; To do that one will have to wait until clickgen supports hyprcursors and I think that it will because the issue for this is opened by the maintainer. |
I think we could get away with something a lot simpler. Hyprcursor provides hyprcursor-util which enables us to convert any xcursor packages to a hyprcursor compatible version. So let's say the user specifies something like: cursor.package = pkgs.bibata-cursors; And then the cursors get installed somewhere like Then we could run: # extract intermediate "working state"
hyprcursor-util -x ~/.icons/bibata-cursors -o ~/.icons/bibata-cursors-temp
# generate the hyprcursor compatible version
hyprcursor-util -c ~/.icons/bibata-cursors-temp -o ~/.icons/bibata-cursors-hypr
# remove leftover intermediate state
rm -rf ~/.icons/bibata-cursor-temp Then we can just tell hyprland to use |
I am not an expert in cursors so correct me if I am wrong. As far as I know:
|
Yes, converting Xcursor to Hyprcursor has no benefits as far as I can see, since Hyprland can load Xcursor directly anyway. Conversion in the other direction could be useful however. See here (taken from #102) for an attempt at supporting different file formats under the same option, with automatic conversion between them - we may want to use something similar here. |
https://wiki.hyprland.org/Hypr-Ecosystem/hyprcursor/
I don't see anything related to hyprcursor here, so I'm opening this
Is there a reason it's not used? Or was this just not added yet
The text was updated successfully, but these errors were encountered: