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: Show name of struct type defined at file scope #1269

Merged
merged 1 commit into from
Jun 26, 2023

Conversation

FnControlOption
Copy link
Contributor

@nektro
Copy link
Contributor

nektro commented Jun 25, 2023

screenshot?

@FnControlOption
Copy link
Contributor Author

Old

old

New

new

@llogick
Copy link
Contributor

llogick commented Jun 25, 2023

It seems to depend on the pub const Name = and @import("Name.zig"); matching, but that's probably true in 99% of the cases. 👍 😀

@FnControlOption
Copy link
Contributor Author

FnControlOption commented Jun 26, 2023

Yeah, this uses the file name since there's no explicit struct declaration, such as const Name = struct { ... };

@leecannon leecannon merged commit 38a12a5 into zigtools:master Jun 26, 2023
acristoffers pushed a commit to acristoffers/zls that referenced this pull request Jun 28, 2023
@acristoffers acristoffers mentioned this pull request Jun 28, 2023
@FnControlOption FnControlOption deleted the hover branch July 2, 2023 13:24
KoltPenny pushed a commit to KoltPenny/zls that referenced this pull request Oct 18, 2023
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.

4 participants