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 more ALE Highlights #135

Merged
merged 1 commit into from
Apr 15, 2019
Merged

Add more ALE Highlights #135

merged 1 commit into from
Apr 15, 2019

Conversation

meck
Copy link
Contributor

@meck meck commented Aug 10, 2018

This adds colors and underlines for ALE Errors and Warnings. Also for language client Neovim than uses the same highlight groups as ALE

Copy link
Contributor

@arcticicestudio arcticicestudio left a comment

Choose a reason for hiding this comment

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

Hi @meck 👋
Thanks for your contributions and your patience 👍

I've been too busy with my "real" life and the development of the official Nord website to work on the massive amount of issues/PRs that were submitted each day for all of Nord's port projects.

Your changes are looking good to me, makes perfectly sense to align the errors and warnings with the gutter signs.

@arcticicestudio arcticicestudio merged commit 9d82b7a into nordtheme:develop Apr 15, 2019
@arcticicestudio arcticicestudio removed their assignment Apr 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants