sts4 icon indicating copy to clipboard operation
sts4 copied to clipboard

vscode-concourse: add dark background to light mode icon

Open bmalehorn opened this issue 3 years ago • 3 comments

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

Follow-up to https://github.com/spring-projects/sts4/pull/839.

before after
image image

bmalehorn avatar Sep 15 '22 03:09 bmalehorn

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 avatar Sep 15 '22 05:09 martinlippert

(and sorry for the confusion)

martinlippert avatar Sep 15 '22 05:09 martinlippert

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 avatar Sep 16 '22 04:09 bmalehorn

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 avatar Nov 17 '22 13:11 martinlippert