Commit f176575
committed
fix(dashboard): restore ResponsiveContainer in ChartContainer
The dashboard's ChartContainer was missing Recharts' ResponsiveContainer
wrapper and the aspect-video CSS class, causing charts to render with no
dimensions. The ChartWrapper was using a cloneElement hack to compensate,
but this broke after tooltip sync changes.
Restore ResponsiveContainer and aspect-video to match the OSS
implementation, and remove the now-unnecessary cloneElement workaround.1 parent 884ad41 commit f176575
2 files changed
Lines changed: 9 additions & 11 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | 2 | | |
3 | | - | |
| 3 | + | |
4 | 4 | | |
5 | 5 | | |
6 | 6 | | |
| |||
346 | 346 | | |
347 | 347 | | |
348 | 348 | | |
349 | | - | |
350 | | - | |
351 | | - | |
352 | | - | |
353 | | - | |
354 | | - | |
355 | 349 | | |
356 | 350 | | |
357 | | - | |
| 351 | + | |
358 | 352 | | |
359 | 353 | | |
360 | 354 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
80 | 80 | | |
81 | 81 | | |
82 | 82 | | |
83 | | - | |
| 83 | + | |
| 84 | + | |
| 85 | + | |
84 | 86 | | |
85 | 87 | | |
86 | 88 | | |
| |||
91 | 93 | | |
92 | 94 | | |
93 | 95 | | |
94 | | - | |
| 96 | + | |
95 | 97 | | |
96 | 98 | | |
97 | 99 | | |
98 | 100 | | |
99 | 101 | | |
100 | | - | |
| 102 | + | |
| 103 | + | |
| 104 | + | |
101 | 105 | | |
102 | 106 | | |
103 | 107 | | |
| |||
0 commit comments