-
Notifications
You must be signed in to change notification settings - Fork 679
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
Hover consistency with Visual Studio #3090
Conversation
@paladique Thanks for the PR! LGTM |
Looks I need to fix a test too! Working on that now. |
@paladique Note that you also need to update
|
Codecov Report
@@ Coverage Diff @@
## master #3090 +/- ##
=======================================
Coverage 89.79% 89.79%
=======================================
Files 59 59
Lines 1588 1588
Branches 89 89
=======================================
Hits 1426 1426
Misses 151 151
Partials 11 11
Continue to review full report at Codecov.
|
Codecov Report
@@ Coverage Diff @@
## master #3090 +/- ##
=======================================
Coverage 89.79% 89.79%
=======================================
Files 59 59
Lines 1588 1588
Branches 89 89
=======================================
Hits 1426 1426
Misses 151 151
Partials 11 11
Continue to review full report at Codecov.
|
@paladique Thank you for your contribution. Looks good to me :) |
@rchande Should we have a changelog entry for this ? |
Fixes #2610
I also changed the display order so that it looks similar to VS's behavior: