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

Comment continuation isn't working anymore? #4728

Closed
JoshuaBatty opened this issue Jul 3, 2023 · 2 comments · Fixed by #4876
Closed

Comment continuation isn't working anymore? #4728

JoshuaBatty opened this issue Jul 3, 2023 · 2 comments · Fixed by #4876
Assignees
Labels
bug Something isn't working language server LSP server

Comments

@JoshuaBatty
Copy link
Member

I'm just getting everything set up on my new machine and noticed that on_enter isn't adding // or /// in the editor even though I have both settings set to on.

Screenshot 2023-07-03 at 12 34 00 pm

Screen Recording 2023-07-03 at 12 41 57 pm

@sdankel not sure if this is happening for you also?

@JoshuaBatty JoshuaBatty added bug Something isn't working language server LSP server labels Jul 3, 2023
@sdankel
Copy link
Member

sdankel commented Jul 3, 2023

This is happening for me too! Doc comment continuation is still working for me, but not comment continuation. Looking into it.

@sdankel
Copy link
Member

sdankel commented Jul 3, 2023

Weird, after updating VSCode, both are now working for me. I plan to replace this with a custom on_enter method implementation to remove the dependency on the client to apply edits which will hopefully also make this more robust.

sdankel added a commit that referenced this issue Jul 28, 2023
## Description

Closes #4728

Moves the on_enter capabilities to a custom LSP method that simply
returns the edits to the client, rather than directly applying the edits
as part of did_change.

With this approach, there is still some lag if you hit enter many times,
but it seems to be working better than before.

## Checklist

- [x] I have linked to any relevant issues.
- [ ] I have commented my code, particularly in hard-to-understand
areas.
- [ ] I have updated the documentation where relevant (API docs, the
reference, and the Sway book).
- [ ] I have added tests that prove my fix is effective or that my
feature works.
- [ ] I have added (or requested a maintainer to add) the necessary
`Breaking*` or `New Feature` labels where relevant.
- [ ] I have done my best to ensure that my PR adheres to [the Fuel Labs
Code Review
Standards](https://github.com/FuelLabs/rfcs/blob/master/text/code-standards/external-contributors.md).
- [ ] I have requested a review from the relevant team or maintainers.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working language server LSP server
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants