Skip to content

fix: make printed page look somewhat decent #527

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

Merged
merged 3 commits into from
Jul 21, 2025

Conversation

Garmelon
Copy link
Contributor

@Garmelon Garmelon commented Jul 4, 2025

The nav bar took up a lot of space even though it wasn't displayed. I also got rid of the header while I was there.

Some code snippets were printed with a scroll bar instead of wrapping them. Also, footnotes can't float in print note and just looked broken, so I redesigned them as in-line info boxes. They still look a little wonky, but they're a lot better than overlapping digits onto text.

These changes are the result of my experiments on zulip: #new members > Printable form of language reference? @ 💬

They should help with #434 if one uses a browser to convert the single-page manual to a PDF.

The nav bar took up a lot of space even though it wasn't displayed. I
also got rid of the header while I was there.

Some code snippets were printed with a scroll bar instead of wrapping
them. Also, footnotes can't float in print note and just looked broken,
so I redesigned them as in-line info boxes. They still look a little
wonky, but they're a lot better than overlapping digits onto text.
Copy link
Contributor

github-actions bot commented Jul 4, 2025

Preview for this PR is ready! 🎉 (also as a proofreading version). built with commit a934aed.

@david-christiansen
Copy link
Collaborator

Let me know when you're ready for a merge - looks like you're still working on it, so I'll hold off for now.

@david-christiansen
Copy link
Collaborator

And thanks, BTW!

@Garmelon
Copy link
Contributor Author

Garmelon commented Jul 4, 2025

Twice so far I've thought I was done, only for idle scrolling through the PDF to reveal some more scroll bars :D

I've looked through it once more and found no more scroll bars. Should be ready to merge now.

@david-christiansen david-christiansen merged commit 2ae52f3 into leanprover:main Jul 21, 2025
9 checks passed
@david-christiansen
Copy link
Collaborator

Thank you very much!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
HTML available HTML has been generated for this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants