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

Can you remove the ring somehow? #120

Open
davidv171 opened this issue Jun 21, 2018 · 6 comments
Open

Can you remove the ring somehow? #120

davidv171 opened this issue Jun 21, 2018 · 6 comments

Comments

@davidv171
Copy link

I'm currently doing a macabre-ish theme so my lock icon is haloofthesun, that has a dark background and a red sign.

https://pumpkinsoup.deviantart.com/art/Halo-of-the-Sun-201083154

But the green circle when I start typing is so ugly and misaligned with the lock icon that I'm really thinking of removing it. How do I do that? I've changed the parametres for the ring colors but I didnt see any effects.

I removed this:

param=("--textcolor=ffffff00" "--insidecolor=ffffff1c" "--ringcolor=ffffff3e" \ "--linecolor=ffffff00" "--keyhlcolor=00000080" "--ringvercolor=00000000" \ "--separatorcolor=22222260" "--insidevercolor=0000001c" \ "--ringwrongcolor=00000055" "--insidewrongcolor=0000001c")

@meskarune
Copy link
Owner

i3lock creates rings as a password indicator. In order to remove them you will need to make their apha channel transparent and thus invisible.

@Boruch-Baum
Copy link
Contributor

I've implemented this in my PR #126 , using a command-line feature now present in i3lock. They now have an option '-u' that does just what @davidv171 wants.

@edoardopigaiani
Copy link

Just a heads up, the issue is still open because @Boruch-Baum and @AladW argued. The thing is, we still don't have a '-u' parameter to pass, which would be really cool.

@AladW
Copy link
Collaborator

AladW commented Aug 19, 2019 via email

@edoardopigaiani
Copy link

I think we should revisit the issue and allow to pass arbitrary arguments for i3lock-color after "--", instead of using it to select the screenshot utility.

On 8/18/19 5:01 PM, Edoardo (chap) wrote: Just a heads up, the issue is still open because @Boruch-Baum https://github.com/Boruch-Baum and @AladW https://github.com/AladW argued. The thing is, we still don't have a '-u' parameter to pass, which would be really cool. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#120?email_source=notifications&email_token=ABKASJGB4BSP3O6BNKVRHWLQFFP4RA5CNFSM4FGHZENKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4RBZLI#issuecomment-522329261>, or mute the thread https://github.com/notifications/unsubscribe-auth/ABKASJG627F5HIR66KLWWWLQFFP4RANCNFSM4FGHZENA.

I agree, but let's see what the maintainer thinks.

@edoardopigaiani
Copy link

In the meantime, I managed to fix the missing -u parameter thing. I modified the i3lock-fancy script, I'm referring to line 13 where the variable i3lock_cmd is set.

I ended up adding the -u parameter there. Everything is working fine.

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

No branches or pull requests

5 participants