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

Selection displayed on incorrect subplot #2262

Closed
paulsputer opened this issue Jan 18, 2018 · 5 comments
Closed

Selection displayed on incorrect subplot #2262

paulsputer opened this issue Jan 18, 2018 · 5 comments
Labels
bug something broken

Comments

@paulsputer
Copy link

When using subplots and performing selections if the selection is not removed by double clicking prior to performing a select on a different subplot the selection is drawn on the previous subplot rather than the current subplot. However, when selecting only horizontal or vertical the handles are shown on the correct subplot. The data returned from the event however is correct.

paulsputer added a commit to paulsputer/plotly.js that referenced this issue Jan 18, 2018
@alexcjohnson
Copy link
Collaborator

Good catch @paulsputer ! Looks though like the problem is a bit deeper than that - points selected in one subplot keep their selected/deselected styling until you explicitly double click in that subplot. I don't think that's what we want, unless you multi-select, right?

subplot selection

cc @etpinard @dfcreative

@paulsputer
Copy link
Author

Thanks @alexcjohnson - Ahh I hadn't noticed that bit, good point. Seems to get a bit funky with multi select too.
multisubplot-multiselect

Along with deselecting points on a normal select, perhaps we need to keep the previous selection visible when performing a multi-select?

Would you recommend that multi-select should also be capable of selecting traces over multiple subplots?

@alexcjohnson
Copy link
Collaborator

I'm not 100% sure what the right behavior is here, it may be that we need several different ways for this to work. For example, if you have independent data in each subplot, it would be nice to be able to select points from different subplots together. But if the subplots are coupled (like a SPLOM showing slices of higher-dimensionality data eg https://rreusser.github.io/plotly-mock-viewer/#18) it can be nice to have one subplot in control of the others - whether we do it automatically or through some layered-on crossfilter functionality. Though even in that case being able to multiselect across subplots could be cool, as a way to select a sub-population that might not be disjoint within any single subplot.

Anyway, it's clear that this is going to be a bit of a complex issue if we want to fully exploit the possibilities. We could certainly start with a bugfix that doesn't allow multi-subplot select, and just clears any selection in other subplots when you start a new selection, along with fixing the transform issue that kicked this off.

paulsputer added a commit to paulsputer/plotly.js that referenced this issue Jan 18, 2018
+ displays selection box on relevant subplot
+ removes selection box and selected points from other subplots
@paulsputer
Copy link
Author

Looks like some delayed push it's already been fixed in ebaf705 and appears to support multiselect across multi-subplot.

@alexcjohnson
Copy link
Collaborator

Very impressive @dfcreative - that's a lot of issues closed in one PR #2258!

I still think there's a discussion to be had about whether we want selection in one subplot to clear selections in another, but that's for another time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug something broken
Projects
None yet
Development

No branches or pull requests

2 participants