Skip to content

Show module haddock documentation on hover for imports - #5124

Merged
VeryMilkyJoe merged 21 commits into
haskell:masterfrom
fabsch225:hover-import-hello
Oct 11, 2026
Merged

VeryMilkyJoe merged 21 commits into
haskell:masterfrom
fabsch225:hover-import-hello

Conversation

@julianjumper

@julianjumper julianjumper commented Oct 10, 2026 •

Copy link
Copy Markdown
Contributor

Show module haddock documentation on hover on import

Closes issue #5023

@julianjumper
julianjumper requested a review from wz1000 as a code owner October 10, 2026 13:05
@julianjumper julianjumper changed the title Show module haddock documentation on hover on import Show module haddock documentation on hover for imports Oct 10, 2026

@VeryMilkyJoe VeryMilkyJoe left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good to me, it would be nice to see a gif of it in action, otherwise you can also come find me at the hls table and show me on your pc 🙃

@fendor
fendor requested a review from VeryMilkyJoe October 11, 2026 10:52

@fendor fendor left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, good job!

@VeryMilkyJoe VeryMilkyJoe left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, just one nit :)

Comment thread ghcide-test/data/hover/hie.yaml Outdated
@@ -1 +1 @@
cradle: {direct: {arguments: ["Foo", "Bar", "GotoHover", "RecordDotSyntax", "GotoImplementation", "Constructors", "ConstructorsLinear"]}}
cradle: {direct: {arguments: ["Foo", "Bar", "GotoHover", "RecordDotSyntax", "GotoImplementation", "Constructors", "ConstructorsLinear", "DocSource", "DochTarget"]}}

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What does this do? It looks like a typo :)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes it is, no idea why this does not affect anything. Fixed it, and the tests still pass.

@fabsch225

Copy link
Copy Markdown
Contributor

Demo Video for Presenting at MuniHac: here

@VeryMilkyJoe
VeryMilkyJoe enabled auto-merge (squash) October 11, 2026 13:04
@VeryMilkyJoe
VeryMilkyJoe merged commit 75109fd into haskell:master Oct 11, 2026
42 checks passed
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