-
-
Notifications
You must be signed in to change notification settings - Fork 336
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
Some hover improvements #1303
Some hover improvements #1303
Conversation
@FnControlOption Looks good, just a few things about omitting container bodies.
const PSE = packed struct(u32) {
f: u32,
};
pub fn main() !void {
var pse: PSE = undefined;
_ = pse;
} |
@FnControlOption I just made a mistake, I forced pushed to your branch thinking I had a different one checked out. So the commits you added after my comment are gone, I hope you have them locally. Sorry about this, I feel like an idiot :( |
No worries at all! I fortunately still have the commits locally, so I'll go ahead and push them again ❤️ |
Show "Go to Foo | Bar" instead of "Go to Foo | Go to Bar"
Before
Omit container body
Closes #1301
Enable hover and Go to References for loop label declaration
Resolve types of primitive values, enum literals, and error values
Enable hover for merged error sets