front: fix when no curves are selected in rolling stock curve component

[Wadjetz]

close #5365

[codecov]

Codecov Report

Merging #5481 (6172c3d) into dev (acea273) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##                dev    #5481      +/-   ##
============================================
- Coverage     19.47%   19.46%   -0.01%     
  Complexity     2320     2320              
============================================
  Files           871      871              
  Lines        104778   104806      +28     
  Branches       2538     2538              
============================================
  Hits          20402    20402              
- Misses        82902    82930      +28     
  Partials       1474     1474              

Flag	Coverage Δ
front	8.08% <0.00%> (-0.01%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files	Coverage Δ
...ules/rollingStock/components/RollingStockCurve.tsx	0.00% <0.00%> (ø)

[SharglutDev]

Lgtm, tested in local and works as expected :)

If you can just remove the uppercase from your commit and it's perfect.

[flomonster]

Shouldn't we see the graph (with scales) without curves when nothing is selected?

[SharglutDev]

I asked that myself but as it's the same behavior in the rolling stock editor, thought it was intended.

But seems a bit weird yes.

[Wadjetz]

This is how the graph lib works, without the data it cannot build the axes or you have to find a way to give it axes by default (I can look at the lib doc to find that)

[Wadjetz]

In the meantime i can keep the axes names

[axrolld]

I agree thath the scales should remain if that's possible.

Also, be careful when sharing screenshots of rolling stocks on github, the data are not public 😬 We're not supposed to identify a rolling stock with its corresponding data.

[clarani]

LGTM, tested ✔️