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

Gill Sans not loading on Macs #82

Open
ivelasq opened this issue Apr 9, 2020 · 6 comments
Open

Gill Sans not loading on Macs #82

ivelasq opened this issue Apr 9, 2020 · 6 comments

Comments

@ivelasq
Copy link
Collaborator

ivelasq commented Apr 9, 2020

Referencing post: data-edu/data-science-in-education#519

Gill Sans MT is a font that is bundled only on some Macs, and {extrafont} doesn't necessarily load it unless it's already on the user's computer. To be able to use theme_dataedu(), some Mac users have to download Gill Sans MT, use extrafont::font_import(), then loadfonts(). Not sure if this is also the case with Windows users.

Need to decide how to fix this in the package, either by directing users to download the font, throwing an informative warning if the font is not installed on their computers and letting users render the plot with the default font, or changing Gill Sans to a font that doesn't have these issues.

@jrosen48
Copy link
Collaborator

thx for cross-reffing this. Is there a good source for downloading Gill Sans MT? I'm having some difficulty but this may be to my "font newbie status" 😂

@ivelasq
Copy link
Collaborator Author

ivelasq commented May 15, 2020

Try this site out!

@restrellado
Copy link
Collaborator

restrellado commented May 17, 2020 via email

@jrosen48
Copy link
Collaborator

Do you have any thoughts on switching to another font (that is more likely to be installed) - or having something nifty like IF gill sans is not installed then use this font instead?

@ivelasq
Copy link
Collaborator Author

ivelasq commented May 29, 2020

Hi @joshuarosenberg - I have used {emojifont} as an example and now Gill Sans MT is included within the package itself. Give it a spin :)

@restrellado
Copy link
Collaborator

@jrosen48 did this work?

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

No branches or pull requests

3 participants