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

Set font to one installed by default #21

Closed
wants to merge 2 commits into from

Conversation

danmaclean
Copy link

Test code failed on my macOS RStudio installation. Error in grid.Call(C_textBounds, as.graphicsAnnot(x$label), x$x, x$y, : polygon edge not found . Traced to uninstalled font. Change to installed Arial font fixes issue.

Test code failed on my macOS RStudio installation. `Error in grid.Call(C_textBounds, as.graphicsAnnot(x$label), x$x, x$y, : polygon edge not found` . Traced to uninstalled font. Change to installed Arial font fixes issue.
Set font to one installed by default
@giocomai
Copy link
Owner

Thanks for this feedback.

The readme already includes a notice:

[all examples in this page will use the Roboto condensed font; if it is not installed, you can use the default sans. See the Troubleshooting section at the bottom of this readme.]

I will consider making it more prominent.

Also, "Arial" is not available on all operating systems by default (notably, not on Linux).

The default font in the package is already a generic "sans". Since I care about the looks, I also care about the font, which is why I used a custom fonts in the examples. I may well update the package to check for font availability and give an informative warning/error. I opened the relevant issue: #22

Again, feedback appreciated.

@danmaclean
Copy link
Author

Thanks for response, I missed the notice in the README. I think it was likely off screen when I cut and paste the example to test the package. I imagine it's hard to prepare for all user's mistakes! Appreciate your willingness to take account of 'busy' users. This is the best of the gantt chart tools I looked at btw!

@giocomai giocomai closed this Jan 23, 2024
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