Skip to content
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

Print stylesheet #3480

Closed
srittau opened this issue Oct 13, 2023 · 2 comments · Fixed by #3486
Closed

Print stylesheet #3480

srittau opened this issue Oct 13, 2023 · 2 comments · Fixed by #3486
Labels
infra Core infrastructure for building and rendering PEPs

Comments

@srittau
Copy link
Contributor

srittau commented Oct 13, 2023

I'm getting old. When reading through or reviewing things, I like to sit in a comfortable chair with coffee or tea, hold a stack of A4 paper with a large font (who needs trees anyway?) and a marker, instead of reading on a tablet (or - horror - my phone). This is what a printed PEP looks like at the moment:

image

I suggest to add some print styles to the PEPs with the following properties:

  • Remove the top bar.
  • Remove the table of contents side bar.
  • (Optional) Expand the table of content by default. (If that's possible with pure CSS, I haven't investigated.)
  • Slightly tweak the font sizes, especially smaller header sizes, possibly centering the main title.

If there's interest, I could prepare a PR.

@hugovk hugovk added the infra Core infrastructure for building and rendering PEPs label Oct 13, 2023
@hugovk
Copy link
Member

hugovk commented Oct 13, 2023

Sounds reasonable to me, who doesn't like a nice cup of tea and a comfy chair.

It could also remove the light/dark theme button at the top right.

@pradyunsg
Copy link
Member

It could also remove the light/dark theme button at the top right.

I think removing the entire top bar is the proposal, which would cover this as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
infra Core infrastructure for building and rendering PEPs
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants