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

Papyrus-standard multi-line comments not currently recognized for Intellisense tooltips #87

Open
FelesNoctis opened this issue Aug 10, 2019 · 1 comment
Assignees
Labels
bug Something isn't working duplicate This issue or pull request already exists

Comments

@FelesNoctis
Copy link

FelesNoctis commented Aug 10, 2019

Describe the bug
Using the standard multi-line format ;/ /;, while properly handled by syntax parsing, doesn't appear in Intellisense tooltips for local variables, properties, events, or functions.

To Reproduce
Steps to reproduce the behavior:

  1. Create any script object that would produce an Intellisense tooptip on hover.
  2. Add a multi-line comment using ;/ /; above said object.
  3. Hover over object name.

Example
Multi-line comment

Suggested Change
As mentioned by Scrivener07 in Discord and by Joel in issue #82 (comment), the current AST may not be robust enough at this time. However, if and when a custom tokenizer/parser is created, this should likely be handled in the same way as Documentation Comments { }, but within the current Single-Line ; section of the Intellisense tooltip instead.

Related Papyrus Documentation
Creation Kit: Script_File_Structure#Comments

Possible Related Issues
#82
#83

@FelesNoctis FelesNoctis added the bug Something isn't working label Aug 10, 2019
@joelday
Copy link
Owner

joelday commented Sep 27, 2023

@nikitalita Ideally, we'll want to have comments parsed as structured trivia that can be associated with a given symbol. No need to parse anything on the inside.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working duplicate This issue or pull request already exists
Projects
None yet
Development

No branches or pull requests

2 participants