[charts] Issue with imports in DrawingProvider

[apedroferreira]

[charts] Issue with imports in DrawingProvider

[Janpot]

Both this issue and #9826 should be fixed by #10272.

The problem of this issue and #9826 is not that we're not providing a commonjs export. It's that we were declaring in the charts that we have a commonjs export while in reality it didn't work. It didn't work because there is no commonjs version of d3. (And we didn't build a workaround for that limitation yet as we want to spend our time building features first and take this opportunity to probe the community to see how big of a problem it would be to just not build that workaround ever)

Since module resolution amongst bundlers and custom setups these days is basically the wild west, some of them get totally confused. #10272 removes that faulty export so that modern enough bundlers don't get confused. I believe we can close these issues until the community makes a convincing case for us to add a non-broken commonjs version. i.e. Where being on a non-standard setup is justified. While there will be commonjs around for a very long time, the ratio commonjs/ESM will only decrease going forward.

(cc @alexfauquette Would it make sense to create a specific label for issues related to the charts ESM setup? It'd make it easier for me to keep tabs on them)

[alexfauquette]

Created charts: ESM for it 👍

[apedroferreira]

Looks like I fixed it with https://www.npmjs.com/package/@originjs/vite-plugin-commonjs.
Let me know if I should close it, I guess that other people could have the same issue so maybe something can be done from the library side.

[oliviertassinari]

So related to #9826.