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

Make colors for swaylock unlock indicator configurable #1078

Closed
embik opened this issue Feb 17, 2017 · 5 comments
Closed

Make colors for swaylock unlock indicator configurable #1078

embik opened this issue Feb 17, 2017 · 5 comments

Comments

@embik
Copy link

embik commented Feb 17, 2017

This seems to be a rather trivial and rather petty request, but I would like to pass parameters to swaylock to change the unlock indicator's colors. My whole desktop follows a certain colorscheme and my lock script is using a blurred version of my screen as swaylock background, thus the unlock indicator looks a bit off. Would be pretty rad to change the default colors when calling swaylock.

I would implement this myself (doesn't seem too hard in principle), but I have never written a single line of C. So, if anyone has the same itch as me, that would be neat.

@Timidger
Copy link
Member

but I have never written a single line of C

Now's the time to learn! It's not a hard language. At worst, if you don't think it's good enough for a PR (even though it probably is) you can change it for yourself.

@ddevault
Copy link
Contributor

Now's the time to learn! It's not a hard language. At worst, if you don't think it's good enough for a PR (even though it probably is) you can change it for yourself.

This. The odds of someone else doing this for you are low and the odds of you succeeding if you try are high.

@embik
Copy link
Author

embik commented Feb 18, 2017

You're right, I'll take a look after my exams are over.

ddevault added a commit that referenced this issue Feb 22, 2017
Feature for #1078: Configurable swaylock colors
@4e554c4c
Copy link
Contributor

@embik With #1081 this should be possible now.

@embik
Copy link
Author

embik commented Feb 22, 2017

Wow, thanks a lot @4e554c4c 👍

@ddevault ddevault mentioned this issue Apr 18, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

4 participants