Fix resizing pie like traces when textinfo is set to "none"

[robbtraister]

If textinfo==="none", the textposition coercion is skipped for both funnelarea and pie traces. However, there is a place in the plot render logic that assumes textposition is defined and is skipped iff textposition==="none" (https://github.com/plotly/plotly.js/blob/master/src/traces/pie/plot.js#L136).

Since the coercion is not enforced and textposition is undefined (not "none"), related text objects which have not been initialized are attempted to be accessed resulting in a null pointer exception (https://github.com/plotly/plotly.js/blob/master/src/traces/pie/plot.js#L511).

I am open to alternative solutions that may include a textinfo==="none" check in the plot render logic, but this approach seemed to be the most comprehensive.

Reproduction example: https://github.com/robbtraister/plotly-npe

[archmoj]

I was not able able to replicate the bug on this codepen.
Please feel free to open a bug report or reopen this pull request, in case you hit it again.
Thank you!

[robbtraister]

@archmoj as I described in the repro repo, you have to mouseover the plot first, then attempt to resize.

[robbtraister]

I do not have permission to reopen this pull request

[archmoj]

@archmoj as I described in the repro repo, you have to mouseover the plot first, then attempt to resize.

Thanks very much for clear bug report. 🙏
Let me have a second look. I would try to include the fix in the next plotly.js minor possibly today.

[archmoj]

Thanks very much for this PR.
💃