-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
plotly_clickannotation event not firing on editable: true
#1750
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
Comments
We apologise if your graphs are currently broken, adding See for more details on the issue. |
Thank you, that fixes the issue with the event firing but is that event dependent upon the editable configuration option not being turned on? I saw you had it disabled in the Codepen. The use case I am trying to accomplish is allow the annotation to be moved and capture the click event to load the currently set properties into an editor the user can use to update the annotation. Seems like this is a one or the other solution. |
Yeah. I noticed that too. I'm not sure if on purpose and/or the desired behavior.
You should listen to |
To be more specific, I use the plotly_clickannotation event to load the properties into the editor and to immediately change the focus so the text in the annotation never shows as editable and the edits instead happen in the form inputs. Plotly_relayout would fire after a change to the plot occurs but I want the event to be captured before any changes are made. Granted I could use plotly_relayout to capture the info when the annotation is moved but the loading of the inputs should not be dependent upon the annotation being repositioned. |
That's an interesting use case, probably not what we envision After thinking about this a little bit longer, I'm going to call this a bug: Changing the issue title accordingly. |
editable: true
Hi @etpinard, is there any update on this? thanks. |
Has something changed with the plotly_clickannotation event? I have tried reverting back to a previous version (1.22.0) and the event is firing but on latest (1.27.0) it is not.
See CodePen for example using latest version
The text was updated successfully, but these errors were encountered: