-
Notifications
You must be signed in to change notification settings - Fork 506
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
Carson/feature/plotly crosstalk #281
Carson/feature/plotly crosstalk #281
Conversation
- crosstalk::dependencies() was renamed to crosstalk::crosstalkLibs() - Also fix tests and metadata
@@ -52,5 +52,6 @@ Suggests: | |||
R6, | |||
RJSONIO | |||
Remotes: | |||
rstudio/crosstalk | |||
rstudio/crosstalk, | |||
jcheng5/plotly@joe/feature/crosstalk | |||
RoxygenNote: 5.0.1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is here just for the time being since we need to access plotly's "global" crosstalk options, as I'm getting here
This PR has merge conflicts with master. Is it possible to rebase this PR on the master ? cc @jcheng5 |
Closing in favor of #346 for a cleaner history and better diffing |
This branches off of
rstudio/leaflet@joe/feature/crosstalk-filter
, and adds the ability to coordinate with plotly's logic for dynamic coloring and persistent selections. Here is an example http://rpubs.com/cpsievert/204407