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

Quick info whitespace and sizing issue #23781

Open
davkean opened this issue Dec 14, 2017 · 4 comments
Open

Quick info whitespace and sizing issue #23781

davkean opened this issue Dec 14, 2017 · 4 comments
Labels
Milestone

Comments

@davkean
Copy link
Member

davkean commented Dec 14, 2017

In metadata as source, QuickInfo has sizing and whitespace issues:

Too much whitespace, and signature is cut off:
image

Signature cut off:
image

No signature at all:
image

@gundermanc
Copy link
Member

gundermanc commented Dec 14, 2017

I just merged in a new quick info implementation, so this might be our regression. I'll investigate this. Which branch of VS is this?

@gundermanc
Copy link
Member

@davkean, do you have repro steps? Interestingly I haven't been able to find a scenario in which this occurs in the public, dogfooding, and d15.6stg builds. Is this a new feature in Roslyn? FWIW, the second image looks almost as if both the collapse region and quick info tips are visible and they're overlapping.

@davkean
Copy link
Member Author

davkean commented Dec 14, 2017

I'm on this build: Version 15.6.0 Preview 1.0 [27204.1.d15.6]

  1. git clone http://github.com/dotnet/project-system
  2. Run build.cmd
  3. Open ProjectSystem.sln
  4. Navigate to NuGetRestorer.cs
  5. On line 170, go to definition of ProjectAsynchronousTasks in this line:
_projectVsServices.Project.Services.ProjectAsynchronousTasks

Hover over the return types of any of the methods in that class. All of them have spacing issues.

@gundermanc
Copy link
Member

Ok, my change isn't in Preview 1 bits, but I'll take a look.

@jcouv jcouv added the Area-IDE label Dec 14, 2017
@dpoeschl dpoeschl added this to the Unknown milestone Dec 20, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants