NotificationsYou must be signed in to change notification settings
Fork33.8k
Star175k

Add setting to control hover position#127609

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Merged

alexdima merged 1 commit intomicrosoft:mainfromTimmmm:hover_position

Oct 27, 2021

Merged

Add setting to control hover position#127609

alexdima merged 1 commit intomicrosoft:mainfromTimmmm:hover_position

Oct 27, 2021

Conversation

Copy link

Contributor

Timmmm commentedJun 30, 2021

This adds a hover setting to control whether the user wants hovers to appear above or below the relevant line, if there's space.

Tested manually, including when the hover is near the bottom of the editor and this option is enabled.

This PRfixes#78560

Add setting to control hover position

ee601d1

This adds a hover setting to control whether the user wants hovers to appear above or below the relevant line, if there's space.Fixesmicrosoft#78560

sbatten assignedalexdima

Jul 1, 2021

Copy link

ContributorAuthor

Timmmm commentedSep 13, 2021

Any chance of a review of this before it gets conflicted to death?

Timmmm mentioned this pull request

Oct 27, 2021

Show tooltip bellow the mouse cursor, instead of above it#30797

Closed

Copy link

steinhh commentedOct 27, 2021•
edited
Loading

This is almost certainly not the right place for this comment, but sorry: It seems like an incredibly small and simple change to review and merge! And it seems like such a no-brainer... most (99%+) programmers write their code starting at the top with the first line, not at the bottom with the last line... so covering up stuff you've just written, above where you're currently writing, seems like the wrong thing (to 99%+ of users).