Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Some folks are using right-click on data points in their apps

See original GitHub issue

… and PR https://github.com/plotly/plotly.js/pull/2087 broke that 💣

See https://community.plot.ly/t/plotly-click-event-stopped-firing-after-update-on-right-click/6353 and https://community.plot.ly/t/annotation-right-click/6352.

What should we do?

Revert #2087 ?
add a config option that allows plotly_click on right-clicks?
anything else?

Issue Analytics

State:
Created 6 years ago
Reactions:5
Comments:16 (9 by maintainers)

Top GitHub Comments

2reactions

etpinardcommented, Oct 27, 2017

right-click on any data point/element - we should emit a plotly_click event and you can look at e.event.button(s) to determine if it was a right click right-click on annotations - plotly_clickannotation, e.event.button(s) right-click on legend (see #65) right-click off of subplots (axes, margins…) - passes through to graphDiv.oncontextmenu? anything else?

I agree with all of these 👍

2reactions

Melmoth-the-Wanderercommented, Oct 19, 2017

It’s already on the production (right-click detection using plotly_click) and would be nice if we aren’t stuck with Plotly 1.30 forever 😃

Config option looks fine for us, however I don’t fully understand the purpose of #2087 . The need for custom contextmenu is obvious, but you can still disable browser’s context menu on plotl’s placeholder and build custom one on right click, as we do. That’s even more powerful if you can use Plotly event with points passed to the handler (necessary for us). Same applies to plotly_onannotationclick etc. This gives us a possibility to create a different contextmenu based on what’s clicked.

Am I missing something important here?