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

Allow to theme the white color in the progress badge #44604

Closed
tweakimp opened this issue Feb 27, 2018 · 10 comments · Fixed by #83008
Closed

Allow to theme the white color in the progress badge #44604

tweakimp opened this issue Feb 27, 2018 · 10 comments · Fixed by #83008
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug install-update VS Code installation and upgrade system issues themes Color theme issues verified Verification succeeded
Milestone

Comments

@tweakimp
Copy link

clock

I am currently building a theme and want to change the color of the clock that is white inside the yelloq circle. It's color is #ffffff and i changed all colorsettings in my theme file that had that hexcode to a different color, but the clock stays at #ffffff.

There is a foreground color option for numbers (and text in general i suppose) that i already changed, it looks like this:

oneoncircle

@vscodebot vscodebot bot added the themes Color theme issues label Feb 27, 2018
@bpasero bpasero added the feature-request Request for new features or functionality label Feb 28, 2018
@bpasero bpasero removed their assignment Feb 28, 2018
@bpasero bpasero changed the title Busy clock icon on settings does not have an option to change color Allow to theme the white color in the progress badge Feb 28, 2018
@bpasero bpasero added this to the February 2018 milestone Feb 28, 2018
@bpasero bpasero self-assigned this Feb 28, 2018
@bpasero bpasero added bug Issue identified by VS Code Team member as probable bug help wanted Issues identified as good community contribution opportunities and removed feature-request Request for new features or functionality labels Feb 28, 2018
@bpasero bpasero removed this from the February 2018 milestone Feb 28, 2018
@bpasero
Copy link
Member

bpasero commented Feb 28, 2018

Happy for help if someone has ideas. The challenge is to make it possible to theme the white background of that progress icon (living in here).

It should leverage the activityBarBadge.foreground for the color that is currently white and activityBarBadge.background for the color that is currently blue.

/cc @aeschli

@tweakimp
Copy link
Author

My proposal:
move away from an image to a unicode character like U+1F552 🕒

@bpasero bpasero added feature-request Request for new features or functionality and removed bug Issue identified by VS Code Team member as probable bug labels Mar 2, 2018
@bpasero bpasero removed their assignment Mar 2, 2018
@raygervais
Copy link
Contributor

raygervais commented Mar 19, 2018

@bpasero Question, how does one make the clock icon appear from repo version of Code? I'm currently working on this, but I need to figure out how to visually see the icon. Ideas?

@bpasero
Copy link
Member

bpasero commented Mar 20, 2018

@raygervais e.g. open a very large git repository and click refresh from the git viewlet. Or install an extension.

@tweakimp
Copy link
Author

tweakimp commented Mar 20, 2018

@raygervais You can also just check for updates via right click on it and it will appear for a short time.

@svipas
Copy link
Contributor

svipas commented Apr 2, 2018

Also, you can download this extension Notification Tester and use "Show Progress Badge" command.

@tweakimp
Copy link
Author

tweakimp commented Jul 4, 2018

Is this still being worked on? "activityBarBadge.foreground" is not doing anything as far as I know.

@balent
Copy link
Contributor

balent commented Oct 23, 2018

I believe it should be possible with few steps:

  1. Convert the SVG icon from base64 to text (xml)
  2. Inline it into the HTML
    1. Maybe with something like badge.innerHtml = "<svg><path fill="#fff""/></svg>"
  3. Use CSS "fill" property to change color of icon to the "activityBarBadge.foreground"

@Tyriar Tyriar added the bug Issue identified by VS Code Team member as probable bug label Jun 11, 2019
@miguelsolorio
Copy link
Contributor

If anyone is interested, here is the code pointer to where the badge indicator is set:

this.badgeContent = dom.append(this.badge, dom.$('.badge-content'));

.monaco-workbench .progress-badge > .badge-content {
background-image: url("data:image/svg+xml;base64,PHN2ZyB4bWxucz0iaHR0cDovL3d3dy53My5vcmcvMjAwMC9zdmciIHdpZHRoPSIxNCIgaGVpZ2h0PSIxNCIgdmlld0JveD0iMiAyIDE0IDE0IiBlbmFibGUtYmFja2dyb3VuZD0ibmV3IDIgMiAxNCAxNCI+PHBhdGggZmlsbD0iI2ZmZiIgZD0iTTkgMTZjLTMuODYgMC03LTMuMTQtNy03czMuMTQtNyA3LTdjMy44NTkgMCA3IDMuMTQxIDcgN3MtMy4xNDEgNy03IDd6bTAtMTIuNmMtMy4wODggMC01LjYgMi41MTMtNS42IDUuNnMyLjUxMiA1LjYgNS42IDUuNiA1LjYtMi41MTIgNS42LTUuNi0yLjUxMi01LjYtNS42LTUuNnptMy44NiA3LjFsLTMuMTYtMS44OTZ2LTMuODA0aC0xLjR2NC41OTZsMy44NCAyLjMwNS43Mi0xLjIwMXoiLz48L3N2Zz4=");
background-position: center center;
background-repeat: no-repeat;

@miguelsolorio miguelsolorio added the install-update VS Code installation and upgrade system issues label Oct 8, 2019
@miguelsolorio miguelsolorio added this to the October 2019 milestone Oct 22, 2019
@miguelsolorio miguelsolorio removed the help wanted Issues identified as good community contribution opportunities label Oct 22, 2019
@miguelsolorio
Copy link
Contributor

Verification steps:

Install the Notification Tester extension and run the Start Progress Badge command and modify the activityBarBadge.foreground color token via:

"workbench.colorCustomizations": {
        "activityBarBadge.foreground": "#ff0000"

}

The icon should now inherit the same color.

@gregvanl gregvanl added the verified Verification succeeded label Oct 30, 2019
@vscodebot vscodebot bot locked and limited conversation to collaborators Dec 6, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug install-update VS Code installation and upgrade system issues themes Color theme issues verified Verification succeeded
Projects
None yet
Development

Successfully merging a pull request may close this issue.

8 participants