plotly_click event doesn't fire when clicking non-root/leaf nodes of sunburst trace

Originally reported in https://github.com/ropensci/plotly/issues/1648

Minimal example (note how clicking on the middle ring doesn’t trigger plotly_click) https://codepen.io/cpsievert/pen/OJJZyrZ

[etpinard]

Personally, I’m very (very) much in favour of 2).

[alexcjohnson]

1. user clicks on the root node OR a leaf node:

• current: fires plotly_click
• wanted: same

1. user clicks on an inner node:

• current: executes plotly_sunburstclick handler, triggers transition
• wanted: executes plotly_sunburstclick handler, fires plotly_click, triggers transition

1. user clicks on an inner node AND had set up a plotly_sunburstclick handler that returns false:

• current: executes plotly_sunburstclick handler, fires plotly_click
• wanted (by AJ 😏 ): executes plotly_sunburstclick hander

Is that correct?

yep, that’s what I had in mind! 🍻

I can’t really imagine placing the transition info inside the points array, so maybe the event data could have three subcontainers when a transition is triggered:

{
  event: { /* instanceof MouseEvent */ }
  points: [{ /* */ }],
  transition: {
     duration: /* value from the constant file */,
     easing: /* value from the constant file */
  }
}

or maybe a just a boolean would suffice 🤔

As a user what I would want to know is not the transition parameters, but: did the root node of the view change, and if so to which node?

Moreover, theses changes are borderline breaking, so at the very least they would have to be released in a minor.

For sure, minor.