Skip to content

feat: plaintext rfc formatting and misc fixes #111

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 7 commits into from
Jul 24, 2025

Conversation

holloway
Copy link
Contributor

@holloway holloway commented Jul 23, 2025

feat

  • # changed to pilcrow ¶ in markdown pages, and the character isn't visible by default.
  • plaintext RFC font size fixes (responsive). See docstring in rfc-plaintext.css for details.
  • xml2rfc font family changes
  • RFC content is indented from the side (previously was too close to the screen size)
  • use the RFC HTML abstract rather than the one from Datatracker

screenshots

mobile
Screenshot_2025-07-24_20-40-51

tablet
Screenshot_2025-07-24_20-39-21

desktop
Screenshot_2025-07-24_20-37-46

to the right of the content, jumps too.

Another complexity is that some RFCs are wider than 80 chars:
https://notes.ietf.org/VE2EoxbfRCm6X7W6NvghAA?view
Copy link
Member

Choose a reason for hiding this comment

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

notes links are not archival (even if we expect it to be relatively long lived) - if this is meant to stay as doc for future devs, we should put it somewhere that is intended to be archival.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ok, I've inlined it

@holloway holloway marked this pull request as ready for review July 24, 2025 08:57
@holloway holloway merged commit 798f9d5 into main Jul 24, 2025
3 checks passed
@holloway holloway deleted the feat-plaintext-rfc-formatting branch July 24, 2025 09:55
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.

3 participants