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
Fixing issue #5350 by moving the 'false' check #6727
Conversation
@alexcjohnson I saw you were active on another recent PR. Any comments for this little one here? I don't see how I can assign reviewers or get a bit of attention for this one here... |
💃 |
@archmoj Any chance you guys can cut a release with this change so that I don't need to pull the master branch into my project? |
We expect to make a new release next week. Thanks for your contribution @andrej-vasilj! |
@andrej-vasilj FYI - We just released https://github.com/plotly/plotly.js/releases/tag/v2.26.1 |
This is the most minimal change that I could think of to maintain all existing functionality but to prevent canceled single click events from canceling the double click as well. This is my first PR to this project so let me know if I'm missing something.
Note that I had to modify one unit test because the way the code was before, it would call the single click handler even for double click events which the unit test took into account for the expected number of clicks.
type: bug