Skip to content

Notebook API: Support Language Providers for Notebook Output Cells #159356

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

Open
pgfearo opened this issue Aug 27, 2022 · 4 comments
Open

Notebook API: Support Language Providers for Notebook Output Cells #159356

pgfearo opened this issue Aug 27, 2022 · 4 comments
Assignees
Labels
feature-request Request for new features or functionality notebook-output
Milestone

Comments

@pgfearo
Copy link

pgfearo commented Aug 27, 2022

Please reinstate support for SemanticTokenProvider and DocumentLinkProvider for JSON Notebook output cells.

In May 2022 these providers worked in an XPath Notebook as expected on a JSON output cell when the output cell was rendering the result of an executed XPath expression that included XML nodes - see screenshot from May 2022:

json-output

Each XPath node in the result was represented as an XPath location with special syntax-highlighting. The highlighted XPath location functioned as a document-link. Clicking on this would select the corresponding node in the XML document open in VS Code.

Here's the code used to register the providers:

vscode.languages.registerDocumentSemanticTokensProvider({ language: 'json', notebookType: 'xbook' }, new XpathResultTokenProvider(), XpathResultTokenProvider.getLegend()),
vscode.languages.registerDocumentLinkProvider({ language: 'json', notebookType: 'xbook' }, new LinkProvider()),

A VS Code update some time after May 2022 caused language providers for JSON output cells to no longer work. There is now no special syntax-highlighting and a ctrl-click on the XPath location has no effect.

Please consider reinstating support for this feature in VS Code. For info, this is issue #10 as submitted on the XPath Notebook repository.

I'm submitting this as a feature request rather than a bug report because it's possible that the (output cell) language provider feature that this Notebook extension exploits was not intentional.

@vscodenpa
Copy link

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.

Happy Coding!

@vscodenpa
Copy link

🙂 This feature request received a sufficient number of community upvotes and we moved it to our backlog. To learn more about how we handle feature requests, please see our documentation.

Happy Coding!

@vscodenpa vscodenpa modified the milestones: Backlog Candidates, Backlog Sep 2, 2022
@pgfearo
Copy link
Author

pgfearo commented Nov 19, 2022

Any chance of some response that isn't from a bot? Even if it's a "won't do" with some rationale?

@psuzzi
Copy link

psuzzi commented Apr 24, 2023

As of April 2023, this is still an open issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request Request for new features or functionality notebook-output
Projects
None yet
Development

No branches or pull requests

4 participants