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

Sans serif pics website #431

Draft
wants to merge 6 commits into
base: main
Choose a base branch
from
Draft

Sans serif pics website #431

wants to merge 6 commits into from

Conversation

cedounet
Copy link
Collaborator

address #418

@hendricius please check as I never got Ruby to work on my machine... Thanks

Flowcharts were already sans-serif but the new one look weird with a
serif font on the website => let's make it sans by default.

For epub this a tough call but let's assume most people use sans there
as well, it's less weird anyway to have sans-serif pictures in a serif
text than the other way around.
@cedounet cedounet marked this pull request as draft January 11, 2025 15:16
cedounet and others added 2 commits January 11, 2025 15:29
This is really a shoot in the dark... I have not tested it in any way.
@hendricius
Copy link
Owner

@cedounet ✅ - btw. I recommend at some point to try Docker 😎 . Just run make website without having to install ruby bloat locally.

@hendricius
Copy link
Owner

image

But that seems not to have fixed the serif issue.

@cedounet
Copy link
Collaborator Author

Thanks for that I will have another go…

and yes I should get on the docker bandwagon with the cool kids…

pgfplots uses math fonts for graphs ticks... and it looks weird when
building sans serif.

Not sure Fira is necessarily a great choice with open-sans but we have
very little math so that should do...
@cedounet
Copy link
Collaborator Author

By the look of it I do not understand how the svg are built…. I will dig but fwiw latest commit makes the plot nice in sans-serif pdf which is progress.

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