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

Remove darkForeground and lightForeground settings #52

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

Remove darkForeground and lightForeground settings #52

musicfuel opened this issue Mar 4, 2019 · 2 comments
Labels
enhancement New feature or request help wanted Extra attention is needed
Milestone

Comments

@musicfuel
Copy link
Contributor

The overrides for peacock.darkForeground and peacock.lightForeground seem as though they may not be useful relative to the expense of having the settings. Now that tinycolor is in use following #48, it may be better to remove the complexity/overhead of the settings and either use the values as defaults or calculate new ones based on the background luminance.

The option to keepForegroundColor as proposed in #27 is somewhat related here. As part of this effort I propose we add that flag to settings in place of these two as a means to override the default Peacock behavior. By using the keepForegroundColor the user takes matters into his or her own hands and Peacock should only adjust backgrounds.

Version Information

macOS 10.14
VS Code 1.31.1 Stable
Peacock 0.6.0

@johnpapa
Copy link
Owner

johnpapa commented Mar 4, 2019

I agree with this 100%

I'm willing to take it on, but I love community contributions too. I recommend we split these into 2 PRs:

  1. removing the dark and light foreground color properties

  2. add the keep foreground color

@johnpapa johnpapa added enhancement New feature or request help wanted Extra attention is needed labels Mar 4, 2019
@johnpapa
Copy link
Owner

johnpapa commented Mar 5, 2019

closed by #53

@johnpapa johnpapa closed this as completed Mar 5, 2019
@musicfuel musicfuel mentioned this issue Mar 5, 2019
8 tasks
@johnpapa johnpapa added this to the 1.0.0 milestone Mar 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants