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

[hover] [wip] First prototype of show documentation on hover #590

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ejgallego
Copy link
Owner

No description provided.

@ejgallego ejgallego force-pushed the doc_hover_plugin branch 2 times, most recently from ac3f6d0 to 1603925 Compare October 26, 2023 14:02
@Alizter
Copy link
Collaborator

Alizter commented Oct 26, 2023

Very cool! This only works with documentation in the same file though right? If we wanted to make it work across files we would need to get better location information of identifiers from Coq.

@ejgallego
Copy link
Owner Author

Very cool! This only works with documentation in the same file though right? If we wanted to make it work across files we would need to get better location information of identifiers from Coq.

Indeed, this will work for other files once we can have the per-theory Toc.t; the good news is that the code here doesn't need update, but still a lot to do to have coq-lsp working on full projects.

This is a first, simple prototype, mainly to showcase.
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.

2 participants