[charts] series not drawing since v6.0.0-alpha.8

[Sozialarchiv]

Duplicates

• I have searched the existing issues

Latest version

• I have tested the latest version

Steps to reproduce 🕹

I used the single chart example from the docs, but it affects all chart types.

import * as React from 'react';
import { BarChart } from '@mui/x-charts/BarChart';

export default function SimpleCharts() {
  return (
    <BarChart
      xAxis={[
        {
          id: 'barCategories',
          data: ['bar A', 'bar B', 'bar C'],
          scaleType: 'band',
        },
      ]}
      series={[
        {
          data: [2, 5, 3],
        },
      ]}
      width={500}
      height={300}
    />
  );
}

Current behavior 😯

v6.0.0-alpha.8 and v6.0.0-alpha.9

Expected behavior 🤔

It works in v6.0.0-alpha.7

Context 🔦

No response

Your environment 🌎

npx @mui/envinfo

  Don't forget to mention which browser you used.
  Output from `npx @mui/envinfo` goes here.

Order ID or Support key 💳 (optional)

No response

[LukasTy]

Hello, I was unable to reproduce your issue in both StackBlitz as well as locally with the latest release. 🤷
Please provide a minimal reproduction test case with the latest version. This would help a lot. 🙏
A live example would be perfect. This StackBlitz template may be a good starting point. Thank you!

[Sozialarchiv]

I am using symfony encore. I will try to create a test case.

[Sozialarchiv]

I found the reason:

I changed tsconfig.js in the following way:

before:

{
  "compilerOptions": {
    "module": "commonjs",

after:

{
  "compilerOptions": {
    "module": "esnext",
    "moduleResolution": "node",

and now it is running also with v6.0.0-alpha.8 and v6.0.0-alpha.9. So I guess a change in .8 made it incompatible with commonjs.

[Sozialarchiv]

If I revert this commit #10111 it is also working with commonjs

[LukasTy]

If I revert this commit #10111 it is also working with commonjs

What if you apply this fix that we are going to release: #10263?

[Sozialarchiv]

Unfortunately it is not solving the issue.

Thank you very much for your support.

[Janpot]

@Sozialarchiv Would you mind sharing a full reproduction? One we can download and run for ourselves? Could you also try it with the changes proposed in #10272?

[Sozialarchiv]

@Janpot Of course. I tried it already with StackBlitz but I could not make. I will create a github repository. I will try to make it as minimal as possible.

[Sozialarchiv]

https://github.com/Sozialarchiv/charts-commonjs-test

[Janpot]

@Sozialarchiv Thank you, this is very helpful! It looks like it works for me after I apply the updates of https://github.com/mui/mui-x/pull/10272/files in ./node_modules/@mui/x-charts/package.json. Could you confirm?

[Sozialarchiv]

It works with #10272.

Thank you very much.

[Sozialarchiv]

And thank you for great library, too. The chart part is a very nice addition and completes the stack in a wonderful way.

[LukasTy]

And thank you for great library, too. The chart part is a very nice addition and completes the stack in a wonderful way.

Thanks and thank you for using the library in such an early stage and helping us solve such issues! 🙏