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

Bracketed-name fields shown as .null by docs #27

Closed
Lyrth opened this issue Dec 30, 2020 · 1 comment
Closed

Bracketed-name fields shown as .null by docs #27

Lyrth opened this issue Dec 30, 2020 · 1 comment
Labels
bug Something isn't working tooltips Pertains to tooltip documentation that displays when you hover over an element.

Comments

@Lyrth
Copy link

Lyrth commented Dec 30, 2020

Environment

name version
IDEA version IdeaIU 2020.3
Luanalysis version 1.2.1-IDEA203
OS Win7

What are the steps to reproduce this issue?

  1. Create a class/shape with bracketed field names, e.g.:
    image
  2. With the sample shape above, try to index members (and assign to them) through an instance of the class/shape, through brackets or dot notation (if possible), then hover over the indexer (example in "What happens?" section)
  3. Additionally, type a. on a new line and look at the suggestions.

What happens?

2: Bracketed fields show as .null
image
H21AZvY
(Field m is correct when hovered, both by doing a.m and a["m"]: shows Test.m)

3: The bracketed fields aren't being suggested
image

What were you expecting to happen?

For 2, should show the field names normally: treating the n field as Test.n, and others as Test[1], Test[true], etc.
For 3, it should at least include n; better if it'd also suggest the non-string-name fields.

Any logs, error output, etc?

None

Any other comments?

Also, field description is placed too near to the left edge of the tooltip, don't know if it's normal.

@Benjamin-Dobell Benjamin-Dobell added bug Something isn't working tooltips Pertains to tooltip documentation that displays when you hover over an element. labels Dec 30, 2020
@Benjamin-Dobell
Copy link
Owner

The primary issue identified is resolved by 2a48c29

I've opened a separate issue (#28) to track completion of (primitive) indexed field.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working tooltips Pertains to tooltip documentation that displays when you hover over an element.
Projects
None yet
Development

No branches or pull requests

2 participants