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

Bump plotly.js version #1178

Merged
merged 16 commits into from
Jan 25, 2018
Merged

Bump plotly.js version #1178

merged 16 commits into from
Jan 25, 2018

Conversation

cpsievert
Copy link
Collaborator

@cpsievert cpsievert commented Jan 11, 2018

Things that need to be done before we can safely update:

  • Integrate selected attribute/api with crosstalk framework.
  • Integrate unselected attribute/api with crosstalk framework (should opacityDim map to unselected.marker.opacity, etc?).
    (UPDATE: I can't think of a good use-case for changing stuff like marker.color, marker.size, or textfont.color for unselected marks, and in fact, I think it just leads to more confusion than usefulness, so I'm not likely going to add integration)
  • Enable persistent selection via 'shift' key (this should work for all valid event types, not just plotly_selected)
  • Does it make sense for selectedpoints to set an initial crosstalk selection (like defaultValues arg in highlight())?
    (UPDATE: It's definitely not straight-forward how one would use selectedpoints in conjunction with crosstalk, and adding support wouldn't enable anything that can't already be done via defaultValues in highlight(), so I'm not sure the dev time this would require is necessarily worth it)
  • Wait until fix for relayout of height/width with scattergl does not update positioning correctly plotly.js#2282 has been included in an official release

@daattali
Copy link

Two more small issues with this PR (they might automatically get fixed when you fix the bigger selection issue)

  1. When the initial dragmode is zoom and I manually click on lasso selection, the plot stretches horizontally a bit (see GIF below)

  2. When the plot initially loads it looks and scales correctly. When the window is resized to be narrower/wider, the points are not longer kept in the proper location. In the GIF below, after making the windows narrower, many points that were originally positive in the X axis moved over to be to the left of the X axis

@cpsievert
Copy link
Collaborator Author

cpsievert commented Jan 23, 2018

@daattali thanks, I've reported a minimal example here -- plotly/plotly.js#2282

@cpsievert cpsievert changed the title V1.32.0 Bump plotly.js version Jan 23, 2018
@cpsievert
Copy link
Collaborator Author

@daattali re-installing this PR should fix at least your 2nd point, let me know if you find anything else!

@daattali
Copy link

Thanks. I'll take your word for it, I'll test it out when PR is ready/merged

@cpsievert cpsievert merged commit 458dfd9 into master Jan 25, 2018
@daattali
Copy link

The second issue I raised is indeed fixed. The first isn't. But a much bigger issue is that selecting points as you did in my app seems to be very broken:

plotlybugs

@cpsievert
Copy link
Collaborator Author

@daattali thanks, that's odd that the positions change (I wasn't able to replicate that bit), but there does indeed seem to be a problem with Plotly.addTraces() that could explain the lack of color change -- plotly/plotly.js#2298

@daattali
Copy link

Try the app that you helped me work on. When I install plotly from 5e4da51 it works perfectly fine, when I install plotly from today, selection is messed up

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.

2 participants