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

Use a red background for the errors #3734

Closed
wants to merge 1 commit into from
Closed

Use a red background for the errors #3734

wants to merge 1 commit into from

Conversation

cricrio
Copy link
Contributor

@cricrio cricrio commented Jan 27, 2018

Related to: #2645

I set the background color of the errors to red.
There is a screenshot to show the changes.

screenshot at 2018-01-27 17 56 12

@tsriram tsriram requested review from jquense and removed request for jquense January 27, 2018 17:31
@gatsbybot
Copy link
Collaborator

Deploy preview for gatsbygram ready!

Built with commit 8534514

https://deploy-preview-3734--gatsbygram.netlify.com

@LekoArts
Copy link
Contributor

I think you should add transparency to it or make it way lighter. You can‘t read the black text

@KyleAMathews
Copy link
Contributor

I think adding --no-colors is a better quick fix. We should revisit console colors so they work better with different colored backgrounds.

@KyleAMathews
Copy link
Contributor

#3755

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

Successfully merging this pull request may close these issues.

4 participants