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

Feature request: function documentation preview #268

Open
developerdavi opened this issue Oct 4, 2022 · 2 comments
Open

Feature request: function documentation preview #268

developerdavi opened this issue Oct 4, 2022 · 2 comments
Labels
blocked-reason:needs-slang-usedef Advanced usedef requires slangs usedef chains status:blocked type:feature New feature or request

Comments

@developerdavi
Copy link

The issue

When you hover above a function/event call you get some description of it on a popup, but you can't see the documented block - if there's any - in the popup as well like we have on other languages.

Example

Consider this function:
image

If we hover the mouse above a function call we will see the popup with the function definition but without the text extracted from the documentation block:
image

In other languages e.g JavaScript the editor will show the documentation extracted from the JSDOC comment:
image

Is that feature already included in the roadmap or is there some on going work already about that? I'm open to help to implement that if nobody is working on it right now.

@kanej
Copy link
Member

kanej commented Oct 6, 2022

That is a great feature and one we want to deliver. The reason we don't show the documentation against the function currently is that our parser ignores comments (it is designed for compilation and comments don't affect that), hence that information is not coming through in our analysis.

We are developing a new parser that will include the documentation information and associate with the function, so this feature is currently blocked behind that.

@developerdavi
Copy link
Author

Got it! That's awesome. Thank you for the feedback :)

@kanej kanej added type:feature New feature or request blocked-reason:needs-slang-usedef Advanced usedef requires slangs usedef chains status:blocked labels Jan 2, 2023
@kanej kanej moved this to Backlog in hardhat-vscode Aug 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked-reason:needs-slang-usedef Advanced usedef requires slangs usedef chains status:blocked type:feature New feature or request
Projects
Status: Backlog
Development

No branches or pull requests

2 participants