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

Issues encountered with new zoom functionality #181

Closed
aderidder opened this issue Jan 14, 2016 · 4 comments
Closed

Issues encountered with new zoom functionality #181

aderidder opened this issue Jan 14, 2016 · 4 comments
Labels
feature something new

Comments

@aderidder
Copy link

Hi,
I'm using the new 'plotly_selected' feature, which is a great improvement. Here are some issues I encountered:

  1. When in Select mode, a 'click' generates undefined eventData
  2. When you're zoomed in, select some points and zoom back out, the selection box does not properly scale, which causes a mismatch between the selected points and the area the selection box shows
  3. When in Select mode, double click removes the selection box, whereas a single-click does not.
    This is the selection:
    image

On a single click it stays the same:
image

When you double-click:
image

In the eventData there is no way distinguish between the two, as both are undefined, whereas, if the selection box is removed, you'd also want to be able to remove the selected points to keep the plot visually consistent.

@etpinard
Copy link
Contributor

@aderidder These are known limitations of our first implementation of the lasso/select-box functionality.

I'll rename the issue accordingly.

You can look at PR #154 to know more about what you can expect in the current plotly.js version.

@etpinard etpinard changed the title Issues encountered with new zoom functionality Make selections movable and resizable via click Jan 14, 2016
@etpinard etpinard added the feature something new label Jan 14, 2016
@etpinard
Copy link
Contributor

More info here: #154 (comment)

@etpinard
Copy link
Contributor

cc @dfcreative @chriddyp @jackwluo

@monfera monfera changed the title Make selections movable and resizable via click Issues encountered with new zoom functionality Nov 3, 2017
@etpinard
Copy link
Contributor

merging with #1851

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

No branches or pull requests

2 participants