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

Add Yellow Markers for Warnings #3

Merged
merged 2 commits into from
Sep 11, 2018
Merged

Add Yellow Markers for Warnings #3

merged 2 commits into from
Sep 11, 2018

Conversation

mattlyons0
Copy link
Contributor

Simply add a different (yellow) colored marker for warnings.

Thank you for this package, its been super useful for me!

Screenshot

@mattlyons0
Copy link
Contributor Author

I'm using ESLint which has both warning and error types but I think its worth considering a way for users to specify types and their corresponding colors (as other linters may have more types)

@mattlyons0
Copy link
Contributor Author

Since you are actually pulling in diagnostics messages I added a Info marker. Additionally an idea at solving surdu/scroll-marker#4 is to add transparency to the markers so overlayed ones will blend

@surdu
Copy link
Owner

surdu commented Sep 10, 2018

Holy cow of coincidence! It appears I was not watching my own repo 😅 and didn't get any notifications of you opening a pull request, just came by the repo with other reasons. Fixed that ...

I thought about this, but I forgot why I decided to go with all red ... 😖

Anyway, if you could please choose a more orange color for the warning layer, to not be confused with search results, I would gladly merge your changes. My suggestion would be #fd8926.

Thank you and sorry of the late response 😞

@mattlyons0
Copy link
Contributor Author

No problem, I updated the color to that.

@surdu surdu merged commit 1e75f1e into surdu:master Sep 11, 2018
@surdu
Copy link
Owner

surdu commented Sep 11, 2018

Thank you for your contribution!

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