Skip to content

Update svglite-based snapshots #6453

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

Closed
wants to merge 1 commit into from

Conversation

teunbrand
Copy link
Collaborator

For a few visual tests, we use a custom svglite writer instead of vdiffr's writer to incorporate patterns.
The latest release of svglite appears to write text slightly differently, which this PR updates.

teunbrand added a commit to teunbrand/ggplot2 that referenced this pull request May 19, 2025
teunbrand added a commit that referenced this pull request May 19, 2025
* deal with NA/NULL breaks earlier

* generalise `trans_support_nbreaks()` --> `support_nbreaks()`

* early exit on zero-range limits

* apply `n.breaks` regarless or whence breaks-function came

* Throw `NA` breaks error in 1 place

* omit outdated comment

* include binned scales

* approve pattern snapshots (re: #6453)

* add news bullet
@teunbrand
Copy link
Collaborator Author

I've piggybacked this into #5974, so this can be closed

@teunbrand teunbrand closed this May 19, 2025
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.

1 participant