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

fix: completions shown if cursor in right position #19

Merged
merged 3 commits into from
Feb 27, 2021

Conversation

alexanderniebuhr
Copy link
Member

check if cursor is in correct position than add completion items otherwise do nothing
closes #2

check if cursor is in correct position than add completionitems otherwise do nothing
@alexanderniebuhr
Copy link
Member Author

works with my tests so far. please have a quick test with "F5" too. thx

@alexanderniebuhr
Copy link
Member Author

@voorjaar can we get high severity on this. as this seems to affect many users. If you approve. I will take heavy lifting of merging and release 💯

@antfu
Copy link
Member

antfu commented Feb 27, 2021

@alexanderniebuhr I can't get it verified on my machine, but it looks ok to me. If you believe it's important, I'd say just go ahead and release it. It's better to have a release that people can revert to old versions if it breaks anything rather than no options to choose.

@alexanderniebuhr alexanderniebuhr merged commit 0c6853b into main Feb 27, 2021
@alexanderniebuhr alexanderniebuhr deleted the fix-completion_triggers branch February 27, 2021 21:28
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

Successfully merging this pull request may close these issues.

autocomplete just on html attributes in svelte
2 participants