Skip to content

Conversation

jasonlevinsohn
Copy link

  • Started using this color and really like it, but the error color when checking validation or syntax is very faint.
  • Error's should stand out
  • Thought others might share the same sentiment

@justinmk
Copy link
Contributor

justinmk commented Jun 5, 2014

Can you post a before/after screenshot?

@jasonlevinsohn
Copy link
Author

Sure.

screen shot 2014-06-05 at 2 15 27 pm

screen shot 2014-06-05 at 2 16 13 pm

@justinmk
Copy link
Contributor

@jasonlevinsohn Could you also add ctermbg and ctermfg for the terminal version?

I would like to see this change merged. The current error color is close to useless.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@justinmk The foreground color I think is fine. I changed the background in the 256-color section to the closest color I could find.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ctermfg=219 is a bit muddled, I would vote to just go for a full ctermfg=255.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@justinmk I couldn't get my editor to display in 256 color so I'm not able to test. updated to ctermfg=255

edentsai added a commit to edentsai/extended-molokai that referenced this pull request Oct 2, 2015
Changed Error color to #ff3333 red, better visibility
edentsai added a commit to edentsai/extended-molokai that referenced this pull request Oct 4, 2015
Changed Error color to #ff3333 red, better visibility
edentsai added a commit to edentsai/extended-molokai that referenced this pull request Oct 4, 2015
Changed Error color to #ff3333 red, better visibility
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.

2 participants