-
Notifications
You must be signed in to change notification settings - Fork 72
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
consolidate Color() calls #1481
Comments
We might want to look at our colors in general: $ rg ' Color\(' --stats --quiet
192 matches
192 matched lines
68 files contained matches
972 files searched We have 192 calls to |
agreed, my single biggest pain point with extending the UI was being able to reach the color variations from the VSKIN -- otherwise i could attach all the parts, i just couldn't style it to part |
Unified the usage of the new spec color in some places I could find. There are probably more, but that's it for now. Fixes #1481
I think this needs more investigation than just the spec color :) |
I renamed the issue to reflect that |
a lot of Color() calls reinvent color values for things which already exist or are so isolated that they can't be reached and shared, so they should be consolidated
The text was updated successfully, but these errors were encountered: