Skip to content

Looks bad with dark Alfred theme #1

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

Closed
valpackett opened this issue Apr 30, 2015 · 6 comments · Fixed by #6
Closed

Looks bad with dark Alfred theme #1

valpackett opened this issue Apr 30, 2015 · 6 comments · Fixed by #6

Comments

@valpackett
Copy link

Hi! Thank you for your workflow, I really needed that :)

I think you need to add white background to the symbols to avoid this:

screen shot 2015-04-30 at 8 28 28 pm

Also, why only prefix matching? I want to find \Rightarrow by searching arrow...

@wookayin
Copy link
Owner

wookayin commented May 1, 2015

Hi @myfreeweb, thanks for the report. For dark themes, it would be better (i) to have inverted colors (i.e. white forecolor with transparent background) or (ii) to add white bakground as suggested by you.

However, as I was using bright themes when first trying to write this workflow, I had thought white background looks a little bit clumsy on bright backgrounds. So I think I have to might detect the background color.

Fortunately, from Alfred 2.4, there is an environment variable alfred_theme_background so it seems that we can detect the background color!

I hope I can add this feature very soon.

@valpackett
Copy link
Author

Well, you can have a white background with rounded corners :D

@wookayin
Copy link
Owner

wookayin commented Nov 9, 2021

Hi @unrelentingtech, do you still need this?

@valpackett
Copy link
Author

haha no, I haven't even used macOS in years

@xiajunwen1007
Copy link

Hi @unrelentingtech, do you still need this?

I need this so much,, do you have any plan of supporting it?

@wookayin
Copy link
Owner

wookayin commented May 2, 2022

I added white icons in #6. Please try out v1.1.0 and thank you for waiting for this feature for a long time!

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 a pull request may close this issue.

3 participants