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 color for menu highlighting on hover. #5

Merged
merged 1 commit into from
Nov 24, 2019

Conversation

sargalias
Copy link
Contributor

Hi Billy,

dracula-at-night is my favorite VSCode theme and the one I use everywhere.

However one thing I find difficult with it is that it doesn't have a hover color for menus, so it's difficult to see what the selection will be. I added a hover color for that. This includes the top menu and context menus from right-clicks. The color I used is the same as the hover color on the top bar menu.

Any comments are welcome and if you don't like the change please let me know.

Before the change: Here I am hovering over the "new terminal" option (the mouse isn't visible on screen captures).
Before

After the change:
After

@felipe1982
Copy link

Better than what we have presently. Much appreciated!

@felipe1982
Copy link

Can we merge this?

@sargalias
Copy link
Contributor Author

@bceskavich should we merge this pull request?

@markey
Copy link

markey commented Nov 23, 2019

Come on, please merge. We love your theme but the menu issue is maddening.
Would make a perfect weekend merge ;)

@bceskavich
Copy link
Owner

Hi everyone! @sargalias @felipe1982 @markey apologies for the radio silence on my end. I'll merge this now and get it out into a release later today.

@bceskavich bceskavich merged commit 46dd8aa into bceskavich:master Nov 24, 2019
@markey
Copy link

markey commented Nov 24, 2019

Fantastic, thanks @bceskavich 🥇

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.

4 participants