Skip to content
This repository has been archived by the owner on Jan 25, 2024. It is now read-only.

Excessive hover info? #68

Open
aaronjanse opened this issue Nov 17, 2021 · 2 comments
Open

Excessive hover info? #68

aaronjanse opened this issue Nov 17, 2021 · 2 comments
Labels
question Further information is requested

Comments

@aaronjanse
Copy link
Member

Right now, hovering over any whitespace or comments in an attribute set shows the value of the entire set. This is great for small sets, but it might be unwanted for large ones (e.g. nixos modules).

I guess my question is: when is it appropriate to show the value of an expression when hovering?

@aaronjanse aaronjanse transferred this issue from nix-community/rnix-parser Nov 17, 2021
@aaronjanse
Copy link
Member Author

I had the wrong repo open while creating this issue, so I've transferred it here from rnix-parser 🤦

@aaronjanse aaronjanse added the question Further information is requested label Nov 17, 2021
@Ma27
Copy link
Member

Ma27 commented Dec 19, 2021

I think it'd be a bit harsh if we only show the attr-set when hovering on the closing/opening bracket in an expression like

{
  foobar = 1;
}

Would it make sense to show an even shorter summary when hovering over comments (or at least whitespace)? Because I can imagine that it's useful to see what "context" you're in when editing lines in a large attr-set, however I'm not much into UX, so I'm not sure :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants