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

Adjust activity bar badge color based on background #58

Closed
musicfuel opened this issue Mar 6, 2019 · 4 comments
Closed

Adjust activity bar badge color based on background #58

musicfuel opened this issue Mar 6, 2019 · 4 comments
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@musicfuel
Copy link
Contributor

The current colorization of the activity bar includes the background, foreground, and inactive background. The badge color that can appear over icons is still being maintained based on whatever the current theme is and is not reactive to the background color of the activity bar itself. This leads to potential usability/readability issues with the badge.

As an example, in my current Atom One Dark theme the badge color is a medium blue of #578afa. When using this in with the React Blue coloring the difference is subtle and difficult to see.

screen shot 2019-03-06 at 9 47 29 am

The problem gets even more noticeable if taking the extreme example where "Surprise Me" happened to land on the same color or even something close to it where the badge is effectively blended in.

screen shot 2019-03-06 at 9 50 28 am

Proposal

Leverage the tinycolor library to make intelligent color calculations to adjust the activity bar badge color to some form of a complementary color when the user has chosen to colorize the activity bar. The same images above with that type of logic might instead look like:

screen shot 2019-03-06 at 1 00 27 pm

screen shot 2019-03-06 at 1 01 59 pm

Version Information

macOS 10.14
VS Code 1.31 stable
Peacock 0.7.0

@johnpapa
Copy link
Owner

johnpapa commented Mar 6, 2019

I agree 100%. I was experimenting with badge - but also don;t want to go too far into intruding on someone's theme. Therefore, I think 2 things may come of this:

  1. Let's move forward with your proposed changes
  2. Drafting guidelines for what Peacock should affect

@musicfuel
Copy link
Contributor Author

Awesome. I have this partially in progress since I was just playing with it a bit. I will continue on that path and submit a PR.

I also agree with the guidelines.

@johnpapa
Copy link
Owner

johnpapa commented Mar 6, 2019

perfect. it's all yours

@johnpapa johnpapa added the enhancement New feature or request label Mar 7, 2019
@johnpapa johnpapa added this to the 1.0.0 milestone Mar 7, 2019
@musicfuel
Copy link
Contributor Author

The roadmap to 1.0 item for this in #56 can be marked done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants