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

feat(highlights): add mason support #31

Merged
merged 1 commit into from
May 31, 2024
Merged

feat(highlights): add mason support #31

merged 1 commit into from
May 31, 2024

Conversation

antoineco
Copy link
Contributor

@antoineco antoineco commented May 22, 2024

Improves the highlighting of Mason windows (a popular installer for external tools such as LSP servers), which is inconsistent with the colorscheme by default.

before

mason_before
mason_g_before

after

Outdated screenshots

mason_after
mason_g_after

@windowsrefund
Copy link
Contributor

Looks great. Looking forward to seeing this merged. By the way, no need to update doc/mellifluous.txt as I learned it is automatically updated.

@antoineco
Copy link
Contributor Author

@windowsrefund thanks for the hint!

@windowsrefund
Copy link
Contributor

@antoineco Thank you for all these PRs. Exciting!

@ramojus
Copy link
Owner

ramojus commented May 30, 2024

Everything is great otherwise, thanks! Sorry for the delay on this one, Mason UI was not working in my setup for some reason.

@antoineco
Copy link
Contributor Author

@ramojus No problem!

Everything is great otherwise

The "otherwise" makes me guess that you had a change in mind, but possibly didn't submit the review comment?

@antoineco
Copy link
Contributor Author

Here is an updated screenshot after addressing the suggestions from the review: Screenshot 2024-05-31 at 15 37 29

@ramojus ramojus merged commit 2f7a501 into ramojus:main May 31, 2024
@antoineco antoineco deleted the feat/mason branch May 31, 2024 15:27
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.

3 participants