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

typescript: autocomplete missing on some things #42

Closed
luke-john opened this issue Aug 7, 2017 · 5 comments
Closed

typescript: autocomplete missing on some things #42

luke-john opened this issue Aug 7, 2017 · 5 comments
Assignees

Comments

@luke-john
Copy link

Expected behaviour:

Glamorous component factory style arguments autocomplete the same as they do in vscode.

vscode-autocomplete

Actual behaviour:

Glamorous component factory style arguments do not autocomplete the same as they do in vscode.

stackblitz-autocomplete

Reproduction:

The screenshot for the stackblitz behaviour was captured from the following project.

https://stackblitz.com/edit/react-ts-u9qiu6?file=components%2FApp%2FApp.glamorous.tsx

@luke-john
Copy link
Author

Possibly it's to do with the version of typescript, Glamorous makes use of some 2.4+ features. I couldn't find what version of typescript stackblitz uses to confirm.

@EricSimons
Copy link
Member

EricSimons commented Aug 7, 2017

Thanks for reporting this (and for the detailed repro)! We're on TS 2.4.1, but I think I might know what's causing this — will dig in tmrw with @apai4 and report back 👍

@luke-john
Copy link
Author

hey @EricSimons I had some time to look into this this morning and worked out this is likely an issue with the version of typescript that monaco is using under the hood.

It appears to be pre 2.2 which is when typescript inroduced support for extending interfaces from types. microsoft/TypeScript#13604

A simple reproduction is available at the following link.

https://stackblitz.com/edit/react-ts-n3tizf?file=index.tsx

So it looks like this may be fixed with the next update to the version of monaco-editor that's being used under the hood by stackblitz.

Notes: monaco editor pulls in typescript via monaco-typescript https://github.com/Microsoft/monaco-typescript/blob/master/package.json

@luke-john
Copy link
Author

Just checked it out and everything now seems to all work great :)

Thanks so much for all the work that's been put into stackblitz 💖

@EricSimons
Copy link
Member

@luke-john I totally forgot to circle back and let you know we bumped the monaco version earlier this week, so sorry about that! 😅 No problem at all, thanks a ton for all of your help in tracking down this bug (+ others) - we seriously appreciate it!!

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

No branches or pull requests

3 participants