ipatalas / vscode-postfix-ts

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

Postfixes should be suggested only after `.` #64

Closed zardoy closed 2 years ago

zardoy commented 2 years ago

For now, postfixes are suggested after = and a few other characters (I can't remember other cases, sorry).

For example:

test.test = new // postfix is suggested, but it shouldn't appear here

Since as I suppose name can only be a single word (this is how builtin snippet works) why don't we check that character before current word is .?

ipatalas commented 2 years ago

Interesting... I didn't notice that before but you're right, it really gets triggered. That should be easy to fix as I already check if the dot exists in that line so it's just a matter of checking if it's on the right place in relation to the cursor.

zardoy commented 2 years ago

Hey, @ipatalas thanks for fixing the issue, but please release this on Marketplace

ipatalas commented 2 years ago

Sorry, didn't make it before holidays. I'll be back next week so please stay tuned.

ipatalas commented 2 years ago

New version published