-
Notifications
You must be signed in to change notification settings - Fork 274
Salsa Doesn't Provide Intellisense When Using Import Statements #61
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
Comments
Fix is out for review. See microsoft/TypeScript#7249 (comment) |
Thanks for keeping us looped in! 👍 |
Looks like the change was merged. I'll verify once we get the typescript bits. |
Any update on when this fix will be pushed out? Negates one of the most awesome features of VSCode, at least for my current project. |
@zchrykng The fix is available in the latest VS Code insiders build. Pls see https://code.visualstudio.com/blogs/2016/02/01/introducing_insiders_build You need to enable "allowSyntheticDefaultImports". This is the jsconfig.json:
{
"compilerOptions": {
"allowSyntheticDefaultImports": true
}
} |
@egamma Awesome. Thanks! Installing now. |
Salsa IntelliSense does not work when using ES6 style import statments. I've tried tweaking some tsconfig.json settings, but without much luck.
This does not work:
import React, { AppRegistry } from 'react-native';
But this does:
var React = require('react-native'); var { AppRegistry } = React;
This is a known TypeScript issue:
microsoft/TypeScript#7000
The text was updated successfully, but these errors were encountered: