chartjs / chartjs-plugin-datalabels

Chart.js plugin to display labels on data elements
https://chartjs-plugin-datalabels.netlify.app
MIT License
863 stars 460 forks source link

Use element width/height in fallback positioner #327

Closed kurkle closed 1 year ago

kurkle commented 1 year ago

At least couple of Chart.js extensions use x, y, width, height props in their elements, where x, y describe the top/left corner.

This change would utilize the width and height to determine the center point of the element.

This would be a breaking change if there is an extension that both has width/height on the elements and uses x/y for center point.

My initial thought was to move the positioners to defaults and use the element id to find from there, but its a bit larger change and would not work with derived element types (instanceof PointElement probably works for derived point elements). Also the defaults do not apply when the plugin is not registered globally.