Skip to content
This repository has been archived by the owner on Jul 31, 2022. It is now read-only.

Use the theme API to set the right colours for the status bar item #422

Merged
merged 2 commits into from
Feb 16, 2022

Conversation

orta
Copy link
Member

@orta orta commented Feb 8, 2022

Avoids:
Screen Shot 2022-02-08 at 12 22 07 PM

Only sets the color when it is an error, and it uses the theme API to grab a warning colour. Unsure if this will be perfect, but bright red wasn't going to work in all cases either 👍🏻

@changeset-bot
Copy link

changeset-bot bot commented Feb 8, 2022

🦋 Changeset detected

Latest commit: 518420d

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
vscode-graphql Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@orta
Copy link
Member Author

orta commented Feb 8, 2022

It also runs prettier on the file because the vscode settings are set to format on save

@acao
Copy link
Member

acao commented Feb 16, 2022

Awesome! Next up, we should make this status bar report config status.

@acao acao merged commit 0e2235d into graphql:master Feb 16, 2022
@github-actions github-actions bot mentioned this pull request Mar 8, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants