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

VS Intellisense for csx files does not support #load #13888

Closed
adamralph opened this issue Sep 18, 2016 · 10 comments · Fixed by #17263
Closed

VS Intellisense for csx files does not support #load #13888

adamralph opened this issue Sep 18, 2016 · 10 comments · Fixed by #17263

Comments

@adamralph
Copy link
Contributor

Version Used:

Visual Studio Enterprise 2015 Update 3.

Steps to Reproduce:

  1. Create a csx file foo.csx containing the definition of a class named Foo.
  2. Create another csx file.
  3. Add a line #load "foo.csx"
  4. On the next line type Foo..

Expected Behavior:

Intellisense is shown for Foo.

Actual Behavior:

Intellisense is not shown for Foo.

@adamralph
Copy link
Contributor Author

🎉

@adamralph
Copy link
Contributor Author

@forki
Copy link

forki commented Mar 3, 2017

One question about syntax coloring. In visualfsharp we noticed that a change in the string literal of a #r and #load triggers reparsing of the script in the ide and intellisense works as expected, but syntax recoloing is not triggered. @vasily-kirichenko and myself believe that a change in a string literal is maybe handled differently by roslyn. Probably in order to be faster in coloring.

@forki
Copy link

forki commented Mar 3, 2017

Here is a video of what I described:

colors

@tmat
Copy link
Member

tmat commented Mar 3, 2017

@KevinRansom

@forki
Copy link

forki commented Mar 3, 2017

@tmat we believe this is a roslyn issue. VisualF# doesn't do much interesting stuff here.

@vasily-kirichenko
Copy link

@forki it's our problem.

@forki
Copy link

forki commented Mar 3, 2017

ok. then forget what I wrote ;-)

@vasily-kirichenko
Copy link

@forki see dotnet/fsharp#2520

@forki
Copy link

forki commented Mar 3, 2017

Ok @vasily-kirichenko fixed it. Sorry for the noise

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants