dotnet / fsharp

The F# compiler, F# core library, F# language service, and F# tooling integration for Visual Studio
https://dotnet.microsoft.com/languages/fsharp
MIT License
3.83k stars 773 forks source link

Intellisense Quick Info blocks half of my screen when I hover over some keywords #2628

Open Soldalma opened 7 years ago

Soldalma commented 7 years ago

I am using VS 2017, programming in F#. When the mouse cursor hovers over Deedle.Frame and a few other keywords the Quick Info popup takes almost half my screen.

Repro steps

  1. Install the Deedle package from NuGet and insert "open Deedle" in your code. Type "Frame" in a blank line.

  2. Hover with the mouse cursor over the word "Frame".

Expected behavior

What I would like to have is either a key combination that would make the pop-up go away, or a configuration options under Tools/Options/Text Editor that would let me disable Quick Info.

Actual behavior

A large Quick Info pop-up takes almost half the screen. If I want to make it disappear I have to move the mouse cursor out of the pop-up area, which is a fairly long distance.

Known workarounds

Provide a description of any known workarounds. One can carefully avoid placing the cursor over certain words, but in practice this is quite inconvenient.

Related information

Provide any related information

cloudRoutine commented 7 years ago

@Soldalma hit ESC

dsyme commented 7 years ago

@vasily-kirichenko Looking at the above - if anything, I think we should add the following to the template text:

If this is a Visual F# Tools bug, please provide an animated screenshot

and give a link to a tool to take an animated screen capture GIF

cartermp commented 7 years ago

I think there's definitely an issue here. You can see it if you hover over other large types, like Async. Personally, I don't see much value in listing so many members for a type, and the large tooltips feel like they get in my way. However, that behavior has existed for a while and certainly some people would prefer it. I think there should be a way to toggle on or off QuickInfo which displays members. I think the default should be not displaying members, but I suppose that would be up for discussion.

cloudRoutine commented 7 years ago

if possible we could take a page out of vscode's book and create tooltip windows that can be scrolled

that way we can restrict the size and stop truncating the content

(with better formatting of course 😉 )

KevinRansom commented 7 years ago

@cloudRoutine It seems like scrolling would be good for tooltips over the an enum or a DU type.

vasily-kirichenko commented 7 years ago

Scrollable (and rather ugly) tooltips has been in Idea for years. I hate them. The info you need is always hidden and you have to scroll.

saul commented 7 years ago

@vasily-kirichenko didn't you fix the huge Async tooltip? I'm 99% sure someone sorted this around the time we introduced syntax highlighted tooltips

vasily-kirichenko commented 7 years ago

@saul yes, @vladima fixed it.

s952163 commented 7 years ago

@Soldalma could you provide a screenshot?

tooltip

I think the issue is that tooltip shows up, you move the mouse, hit escape, and as long as you're on the same word it reappears.

vasily-kirichenko commented 7 years ago

you could restrict width of tooltips to, say, 70% of VS main window size. and 50% of its height. thoughts?

smoothdeveloper commented 7 years ago

@vasily-kirichenko tooltip can overflow the main window, VS is not always maximized.

On small screen though, the tooltip would flash if there is no room for it so there is something we need to figure out still.

s952163 commented 7 years ago

@vasily-kirichenko

you could restrict width of tooltips to, say, 70% of VS main window size. and 50% of its height. thoughts?

Yeah, on second thought restricting the width somewhat would make sense.

I don't particularly mind the current behavior. It seems though there are two possible usability enhancements: 1) restrict the size of the tooltip, and make it expandable/scrollable (:scream cat:) when there is more text 2) make it possible to toggle the appearance/disappearance with some keyboard shortcut or menu setting in options.

@Soldalma any thoughts.

smoothdeveloper commented 7 years ago
type FSharpInfoTooltipSettings = {
    maxWidthPercent        : int option
    maxHeightPercent       : int option
    eachParameterOnOwnLine : bool
    scrollable             : bool
    resizable              : bool
}

Let's not make change without having choice to configure

majocha commented 7 years ago

Another offender: tooltip That strange ellipsis in the middle looks like layout bug.

Soldalma commented 7 years ago

Both alternatives are good, although I would prefer the second one. There is a third alternative: making the tooltip go away with a key combination and not come back until the mouse hovers over a different keyword.

On Fri, Mar 17, 2017 at 11:33 PM, s952163 notifications@github.com wrote:

@vasily-kirichenko https://github.com/vasily-kirichenko

you could restrict width of tooltips to, say, 70% of VS main window size. and 50% of its height. thoughts?

I don't particularly mind the current behavior. It seems though there are two possible usability enhancements: 1) restrict the size of the tooltip, and make it expandable/scrollable (:scream cat:) when there is more text 2) make it possible to toggle the appearance/disappearance with some keyboard shortcut or menu setting in options.

@Soldalma https://github.com/Soldalma any thoughts.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/Microsoft/visualfsharp/issues/2628#issuecomment-287510542, or mute the thread https://github.com/notifications/unsubscribe-auth/AD6zWCVXZA0WxbIYLpM0VsK-AkRVkMRIks5rm0JugaJpZM4MfnsU .

Soldalma commented 7 years ago

I have attached a screenshot. For some reason I could not paste the image on the email.

On Fri, Mar 17, 2017 at 11:33 PM, s952163 notifications@github.com wrote:

@vasily-kirichenko https://github.com/vasily-kirichenko

you could restrict width of tooltips to, say, 70% of VS main window size. and 50% of its height. thoughts?

I don't particularly mind the current behavior. It seems though there are two possible usability enhancements: 1) restrict the size of the tooltip, and make it expandable/scrollable (:scream cat:) when there is more text 2) make it possible to toggle the appearance/disappearance with some keyboard shortcut or menu setting in options.

@Soldalma https://github.com/Soldalma any thoughts.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/Microsoft/visualfsharp/issues/2628#issuecomment-287510542, or mute the thread https://github.com/notifications/unsubscribe-auth/AD6zWCVXZA0WxbIYLpM0VsK-AkRVkMRIks5rm0JugaJpZM4MfnsU .