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

Add warning when size is too small #88

Merged
merged 3 commits into from
Nov 15, 2023
Merged

Add warning when size is too small #88

merged 3 commits into from
Nov 15, 2023

Conversation

willgearty
Copy link
Collaborator

@willgearty willgearty commented Sep 27, 2023

This throws a warning if the specified (or default) size is too small compared to the limits of the y axis (less than 1/1000th). This is mostly for when geom_phylopic is used with coord_sf and the data is projected/transformed.

I've also added an example to the vignette that showcases using coord_sf and a projection.

Fixes #86.

@willgearty willgearty added this to the 1.3.0 milestone Sep 27, 2023
@willgearty willgearty self-assigned this Sep 27, 2023
@willgearty willgearty changed the title Add warning when size is too small Add warning when size is too small (and updates for Cairo/librsvg changes) Oct 10, 2023
@willgearty willgearty changed the title Add warning when size is too small (and updates for Cairo/librsvg changes) Add warning when size is too small Oct 10, 2023
@willgearty
Copy link
Collaborator Author

This is now updated for the precompiled vignettes.

Copy link
Collaborator

@LewisAJones LewisAJones left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @willgearty, good job, this seems to work great. Some very minor comments/suggestions!

R/geom_phylopic.R Outdated Show resolved Hide resolved
vignettes/ggplot-geog-plot-3-1.png Outdated Show resolved Hide resolved
Co-authored-by: Lewis A. Jones <41071747+LewisAJones@users.noreply.github.com>
@willgearty willgearty merged commit 9537b39 into main Nov 15, 2023
8 checks passed
@willgearty willgearty deleted the sf-size branch November 15, 2023 14:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

coord_sf and silhouette size
2 participants