Skip to content
This repository has been archived by the owner on Apr 2, 2020. It is now read-only.

"LSP Snippets" option for completion API #1296

Open
wants to merge 2 commits into
base: tensorflow
Choose a base branch
from

Conversation

marcrasi
Copy link
Collaborator

This adds an "LSP Snippets" option to the completion API, documented in SBCompletionOptions.i.

Basically, these allow us to insert placeholders for arguments when the user selects a function from the autocomplete menu. Monaco-based editors automatically highlight the placeholders and let the user tab between them, so that it's easy for them to fill in all the arguments. It looks like this:
lsp_snippet_placeholders

@marcrasi marcrasi requested a review from bgogul February 14, 2019 01:50
Copy link
Collaborator

@bgogul bgogul left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@marcrasi
Copy link
Collaborator Author

fyi, I'm probably not going to get around to fully releasing this for the next few weeks, so I will just leave this PR open here until I have time.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants