-
Notifications
You must be signed in to change notification settings - Fork 56
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
feat(home-manager): add support for hyprlock #330
Conversation
Fixes #295 Config has been tested and is working |
Oops, just realised this doesn’t actually use the hyprlock port at all, so I need to remove that from sources.json The hyprlock port is only a configuration file? Why does it even exist when the hyprland port is all you need? IMO personalised configurations should not be included in the ctp org, it doesn’t even include the colour variables - wouldn’t it be better suited under |
959f4bd
to
4a6e249
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This module should be enabled in tests/home.nix
Should I disable the test in https://github.com/catppuccin/nix/blob/45745fe5960acaefef2b60f3455bcac6a0ca6bc9/tests/darwin.nix ? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry for the wait. Thanks!
Almost identical to the hyprland home manager module, links the catppuccin source file so colour variables can be used in the config.
https://github.com/catppuccin/hyprlock
Thank you to @PerchunPak for her work in #293