ipatalas / vscode-postfix-ts

Postfix notation for TypeScript/Javascript - extension for VS Code
MIT License
158 stars 43 forks source link

make code fancy and suggestions distinguishable #54

Closed zardoy closed 2 years ago

zardoy commented 2 years ago

This is just a quick fixup. This PR includes only visual changes, hope to see them soon, as they're extremely important for me.

The completion constructor needed latest version of vscode typings so I added @types/vscode and removed the postinstall hook that was responsible for fetching typings.

Screenshot 2021-12-15 213740

The only downside of using syntax highlighting is that we losing word wrap, but I don't think it rly matters since it works only with builtin snippets

ipatalas commented 2 years ago

Hi! That looks really great. I haven't been around for a while but I'm back and keen to release a new version including your changes :) Can you please rebase your PR with most recent changes? I did some updates in result of merging one of previous PRs and there are quite some conflicts.

zardoy commented 2 years ago

Hey! I just rebased it on the lateset changes from develop, but it seems already was merged. If it really was I think we can close this PR.

ipatalas commented 2 years ago

Yeah, I managed to merge it manually by cherry-picking your commit as there's been some noise as well which I did not want to merge.