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

vscode-concourse: add dark background to light mode icon #845

Merged

Conversation

bmalehorn
Copy link
Contributor

@bmalehorn bmalehorn commented Sep 15, 2022

Add a new icon with a dark foreground, to be displayed on light color themes.

Follow-up to #839.

before after
image image

@martinlippert
Copy link
Member

Hey @bmalehorn, thanks a lot for jumping on this. I meant a dark FOREGROUND and a transparent background, much in the same way than the dark-mode-icon has a white foreground and a transparent background. What do you think?

@martinlippert
Copy link
Member

(and sorry for the confusion)

@bmalehorn
Copy link
Contributor Author

Hey @bmalehorn, thanks a lot for jumping on this. I meant a dark FOREGROUND and a transparent background, much in the same way than the dark-mode-icon has a white foreground and a transparent background. What do you think?

Ah no worries. I've updated the PR to have a dark icon on transparent background. But come to think of it, I kind of prefer the white-on-blue icon, which is the only official icon colors that you'll see on https://concourse-ci.org/. I like icons to have a bit of color so they're easier to pick out from each other. So maybe things can be left as-is?

light mode: white on blue, like https://concourse-ci.org/
dark mode: white on transparent, like a hosted concourse instance

I don't have a super strong preference either way and just like having any icons at all.

@bmalehorn bmalehorn closed this Oct 14, 2022
@martinlippert martinlippert reopened this Nov 17, 2022
@martinlippert martinlippert merged commit 2dc31e4 into spring-projects:main Nov 17, 2022
@martinlippert
Copy link
Member

finally picked the new icon-black for the light mode here, looks nicely consistent now between light and dark theme. Thanks again @bmalehorn for contributing this!!!

@martinlippert martinlippert added this to the 4.17.0.RELEASE milestone Nov 17, 2022
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 this pull request may close these issues.

2 participants