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

fixup font issue [INFRA-293] #858

Merged
merged 4 commits into from
Sep 22, 2020
Merged

fixup font issue [INFRA-293] #858

merged 4 commits into from
Sep 22, 2020

Conversation

denniszollo
Copy link
Contributor

@denniszollo denniszollo commented Sep 18, 2020

The 45 degree rotated text apparently was some kind of bitmap font and was rendering poorly on some systems. Removing cmbright fixes. Note, PDF doc may look and feel different after.

@denniszollo
Copy link
Contributor Author

/cc @silverjam

@silverjam
Copy link
Contributor

Can you please check if the pdf generates successfully inside the libsbp-build docket container? I think the cmbright font is what was causing issues last time, but would be good to double check.

@silverjam silverjam changed the title fixup font issue fixup font issue [INFRA-293] Sep 21, 2020
@silverjam
Copy link
Contributor

tried to test this but I'm blocked waiting for #853

@silverjam
Copy link
Contributor

@denniszollo I was seeing the same issue on the docker environment, picking a font seems to resolve the issue, I used the STIX font based on the recommendation from this web page: https://graphicdesign.stackexchange.com/a/517

@silverjam
Copy link
Contributor

Example of how the tables render now:

Screenshot 2020-09-21 135601

@silverjam
Copy link
Contributor

Sometimes text will overlap, but it's still readable:

Screenshot 2020-09-21 141055

@silverjam silverjam merged commit e656768 into master Sep 22, 2020
@silverjam silverjam deleted the dzollo/font_fix branch September 22, 2020 18:26
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.

2 participants