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

Autocomplete causes cursor to jump when adding first explicit import #96

Closed
natefaubion opened this issue Feb 19, 2018 · 10 comments · Fixed by nwolverson/purescript-language-server#24

Comments

@natefaubion
Copy link

vscode-cursor-jump-import

Only seems to happen on adding the first explicit import. After that it works fine.

@nwolverson
Copy link
Owner

Can you try this with some non-comment text at the bottom of the file?

@natefaubion
Copy link
Author

It happens regardless for me. I just used the comment to make it obvious.

@natefaubion
Copy link
Author

Oh maybe it’s only with white space or comments, you mean?

@nwolverson
Copy link
Owner

Well nothing works right at the end of files due to compiler positions thing :) But I don't see that this can be related. Don't worry, I know where to look.

@natefaubion
Copy link
Author

Yeah, it always happens to me, even if there's other content.

@nwolverson
Copy link
Owner

I can reproduce this. Incidentally happens every time if I have a CRLF file on mac, possibly some whitespace nonsense again...

@natefaubion
Copy link
Author

This is better in the new release since it doesn't jump to the end of the line, but it still inserts a newline.

import-newline

@nwolverson
Copy link
Owner

Newline after map in the above? You're just selecting the completion option and that appears?

@natefaubion
Copy link
Author

Yes. Happens on first import. Subsequent imports don’t add a new line.

@nwolverson nwolverson reopened this Mar 2, 2018
nwolverson added a commit to nwolverson/purescript-language-server that referenced this issue Mar 2, 2018
@nwolverson
Copy link
Owner

I believe this is more fixed now.

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 a pull request may close this issue.

2 participants