Skip to content

Animint fun#171

Open
tdhock wants to merge 6 commits intomasterfrom
animint-fun
Open

Animint fun#171
tdhock wants to merge 6 commits intomasterfrom
animint-fun

Conversation

@tdhock
Copy link
Copy Markdown
Owner

@tdhock tdhock commented Nov 2, 2016

rather than explicitly writing structure(list(...), class="animint") we can write animint(...)

structure(q2, class = "animint")
animint(q = qplot(
x, y, data = dat, colour = label,
xlab = "first plot with color legend"))
Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

here I add a test for the new functionality in the existing test for rendering Rmd

@tdhock
Copy link
Copy Markdown
Owner Author

tdhock commented Nov 3, 2016

hey @cpsievert any comments about this new functionality? I remember during your GSOC you said that you would rather be explicit and use structure to attach a class to an animint viz list. However, during my interactive graphics tutorial at useR2016, one of the people suggested to have a function like this.

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