Consolidate typescript language info into one item #133731
Assignees
Labels
bug
Issue identified by VS Code Team member as probable bug
insiders-released
Patch has been released in VS Code Insiders
typescript
Typescript support issues
verified
Verification succeeded
Milestone
Comments
mjbvz
added a commit
that referenced
this issue
Sep 27, 2021
For #133731 No need to show this normally since project intellisense is the default
|
I've pushed a change to only show this when you are single file mode as it's not typically useful otherwise |
|
@jrieken I've consolidated this to a single When you open a file that is in partial mode, we now show:
Otherwise we show the tsconfig (or tell the user that they don't have a tsconfig) Please give this a try in the next insiders and let me know if you have any other suggestions |
|
Looking much better but I am still no super-fan of the dedicated tsconfig-entry. It seems a bit thin and could be merge with the version. It's also unlikely that I would pin those - they also seem to be too generic for that and I wonder if API to suppress pinning is needed? |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
and others
Having three items looks quite verbose and it is hard to tell what belongs together and what not (we should maybe add the source of the message but I don't believe it will make things nicer in this case)
I would merge the information of the upper two TS items into the first, e.g have the version information followed by the project and support-depth information. Note that you use markdown-links for further commands.
The text was updated successfully, but these errors were encountered: