8000 Handle recent changes to ggplot2's `plot_build()` logic by cpsievert · Pull Request #2262 · plotly/plotly.R · GitHub
[go: up one dir, main page]
Skip to content

Handle recent changes to ggplot2's plot_build() logic #2262

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.

Sign up for GitHub

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

Merged
merged 10 commits into from
May 5, 2023
Merged

Handle recent changes to ggplot2's plot_build() logic #2262

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
8a94f66
Close #2259: handle recent changes to ggplot2's plot_build() logic
cpsievert May 2, 2023
0b1fbe9
Apply suggestions from code review
cpsievert May 3, 2023
8f79eab
Try using latest ubuntu
cpsievert May 3, 2023
5b73531
Also accomodate for guide changes made in https://github.com/tidyvers…
cpsievert May 4, 2023
7b34d73
Approve new visual baseline
cpsievert May 4, 2023
98becae
Update test expectations
cpsievert May 4, 2023
8a3d09b
(Temporarily) test with ggplot2
cpsievert May 4, 2023
33ca9bd
Fix more changes in colorscale parameters
cpsievert May 4, 2023
3666c2e
Fix support for no guide
cpsievert May 5, 2023
de2c37b
Accept new visual baselines
cpsievert May 5, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix support for no guide
  • Loading branch information
@cpsievert
cpsievert committed May 5, 2023
commit 3666c2ecc9fcf9f44f0abab4edb201170ffcaee8
3 changes: 2 additions & 1 deletion R/layers2traces.R
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -103,7 +103,8 @@ layers2traces <- function(data, prestats_data, layout, p) {
# now to the actual layer -> trace conversion
trace.list <- list()

aes_no_guide <- names(p$guides)[vapply(p$guides, identical, logical(1), "none")]
guides <- if (inherits(p$guides, "ggproto")) p$guides$guides else p$guides
aes_no_guide <- names(guides)[vapply(guides, identical, logical(1), "none")]

for (i in seq_along(datz)) {
d <- datz[[i]]
Expand Down
0