-
-
Notifications
You must be signed in to change notification settings - Fork 401
4416 - show versions of installed packages #4651
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
base: master
Are you sure you want to change the base?
Conversation
Personally I'd be somewhat inclined to do this in a hover rather than an inlay hint, but debatable. |
Not sure what a coherent argument would look like, but I would argue something like:
|
I added installed package version info to dependency hover and adjusted documentation link to point to the right version as well: Screen.Recording.2025-08-10.at.22.50.48.movFair points and in principle I share your views on inlays. However I still like their look and feel for package versions. You immediately get a list of what's exactly installed. It didn't feel like cluttering the UI, since there's not a lot of LSP related info in Anyway, I'm not married to the idea, so I can as well just drop it (inlays/code lenses) 😉 |
Took my chances on implementing @fendor suggestions from @lucalabs-de #4430 MR.
Inlay Hints

CodeLenses got hard to read when there are multiple packages in single line, so I added package name then to make it easier to understand.

As noticed in previous MR, because of limited positioning options lenses can look a bit wonky depending on cabal file formatting. Hover could be a nice (and easy to add) fallback as well, but I'm not sure about status of hover ticket (#4417) and for now I just decided to avoid unnecessary conflicts.
Closes #4416 (based on #4430)