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

The layout of the graph within the draft in HTML/PDF version is not right #5079

Closed
1 task done
AijunWang opened this issue Feb 3, 2023 · 7 comments · Fixed by #5098
Closed
1 task done

The layout of the graph within the draft in HTML/PDF version is not right #5079

AijunWang opened this issue Feb 3, 2023 · 7 comments · Fixed by #5098
Assignees
Labels
bug Something isn't working

Comments

@AijunWang
Copy link

Describe the issue

In the newest version of HTM/PDF layout of the internet draft, the graph is wrapped around automatically, and let almost every graph can't be easily viewed by the reader.
Can you fix it, or add horizon scroll bar to improve the overview of the draft.

you can check the problem in the "Table of Contents" part of https://datatracker.ietf.org/doc/html/draft-wang-tcpm-tcp-service-affinity-option, and Figure 1 in the introduction part of the same draft.

Code of Conduct

@AijunWang AijunWang added the bug Something isn't working label Feb 3, 2023
@larseggert
Copy link
Collaborator

I see the issue, but I can't reproduce it in my development instance - the PDF looks fine there. @rjsparks, any ideas?

@larseggert larseggert self-assigned this Feb 3, 2023
@rjsparks
Copy link
Member

rjsparks commented Feb 6, 2023

Maybe the differences are in the fonts weasyprint is finding on ietfa vs the dev instances? Note that the header is wrapped badly in the pdfized text cached at the datatracker.

@rjsparks
Copy link
Member

rjsparks commented Feb 6, 2023

The datatracker generated pdfs are using Liberation-Mono - the dev instance generated pdfs are using DejaVu-Sans-Mono

@larseggert
Copy link
Collaborator

That is probably it. We should standardize on fonts asap.

@larseggert
Copy link
Collaborator

My local dev instance also uses Liberation-Mono, so that's not it.

@AijunWang
Copy link
Author

AijunWang commented Feb 8, 2023 via email

@larseggert
Copy link
Collaborator

The fix hasn't been deployed yet.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 17, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants