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

fix(hl): use NormalFloat instead of Normal #591

Merged
merged 1 commit into from
Jul 24, 2022

Conversation

seblj
Copy link
Contributor

@seblj seblj commented Jul 14, 2022

I think NormalFloat is the one to highlight with since it's highlight inside a popup window. I have different backgrounds for a floating window and using Normal looks a bit weird.

Before:
image

After:
image

@seblj
Copy link
Contributor Author

seblj commented Jul 14, 2022

I see CI failed, but it is the same test failing as the one that failed on the latest master, so it's not related to this

@seblj
Copy link
Contributor Author

seblj commented Jul 20, 2022

What do you think? @lewis6991

@lewis6991
Copy link
Owner

Sorry, haven't had a chance to look into this yet. Very busy atm, but will try and get to this when I can

@seblj
Copy link
Contributor Author

seblj commented Jul 20, 2022

Totally understand!

@lewis6991 lewis6991 merged commit 48ac827 into lewis6991:main Jul 24, 2022
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