-
Notifications
You must be signed in to change notification settings - Fork 42
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
Binder symbols in Exists is not linked #91
Comments
This does most likely require a patch to lean itself, doc-gen4 merely analyses datastructures on top of the info trees: https://github.com/leanprover/doc-gen4/blob/main/DocGen4/Output/Base.lean#L208 CC: @Kha what do you think? |
Yes, doc-gen shows whatever the info view shows |
leanprover/lean4#6704 has a possible fix for this. It modifies the app unexpanders for core notations, adding tags to tokens for the head constant. @Komyyy's linked pull request solves this in a different way, by using heuristics that replace using |
This PR is closed because of some problems, leanprover/lean4#6704 will solve this issue. |
https://leanprover-community.github.io/mathlib4_docs/Init/PropLemmas.html#exists_eq has a link now on the exists notation. (App unexpanders in general now have links in their tokens.) |
Compare:
exists_eq
exists_eq
Only the Lean3 versions let you inspect the binder symbol in the docs. While for
Exists
s this is obvious, for other binders the hover text is very useful.It's possible this requires a patch to Lean4 itself; the Lean 3 patch was leanprover-community/lean#781.
The text was updated successfully, but these errors were encountered: