Skip to content

Commit 6847b58

Browse files
committed
Fix typo and chart explorer titles
1 parent 14d353e commit 6847b58

File tree

2 files changed

+13
-17
lines changed

2 files changed

+13
-17
lines changed

frontend/scripts/components/charts/ChartExplorer.tsx

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -79,10 +79,6 @@ const ChartContainer = styled.div`
7979
min-height: 450px;
8080
`;
8181

82-
const TableSection = styled.div`
83-
margin-top: 1.5rem;
84-
`;
85-
8682
const Separator = styled.div`
8783
margin-top: 1.5rem;
8884
padding-top: 1.25rem;
@@ -205,24 +201,25 @@ const ChartExplorer: React.FC<ChartExplorerProps> = ({
205201
</ChartContainer>
206202

207203
{hasTable && (
208-
<TableSection>
209-
<h6 className="fr-mb-1w">Données du graphique</h6>
204+
<Separator>
205+
<h6>Données du graphique</h6>
210206
<ChartDataTable
211207
data={dataTable}
212208
title={chartTitle || ''}
213209
/>
214-
</TableSection>
210+
</Separator>
215211
)}
216212

217213
{sources.length > 0 && (
218214
<Separator>
215+
<h6>Sources</h6>
219216
<ChartDataSource sources={sources} displayMode="text" />
220217
</Separator>
221218
)}
222219

223220
{children && (
224221
<Separator>
225-
<h6 className="fr-mb-1w">Détails du calcul</h6>
222+
<h6>Détails du calcul</h6>
226223
{children}
227224
</Separator>
228225
)}

frontend/scripts/components/pages/Friches/components/FrichesCharts.tsx

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -37,10 +37,9 @@ const DetailsFricheZonageEnvironnemental: React.FC = () => (
3737

3838
const DetailsFricheBySize: React.FC = () => (
3939
<div>
40-
<h6 className="fr-mb-1w">Calcul</h6>
4140
<p className="fr-text--xs">
4241
Les 4 catégories de taille sont déterminées à partir de l'ensemble des
43-
tailles des friches à l'échelle nationale, d'après les friches incluent
42+
tailles des friches à l'échelle nationale, d'après les friches incluses
4443
dans les données <strong>Cartofriches</strong>.
4544
</p>
4645
</div>
@@ -69,6 +68,12 @@ const DetailsFricheByZonageType: React.FC = () => (
6968
</div>
7069
);
7170

71+
const FRICHE_DETAILS: Partial<Record<string, React.ReactNode>> = {
72+
friche_zonage_environnemental: <DetailsFricheZonageEnvironnemental />,
73+
friche_surface: <DetailsFricheBySize />,
74+
friche_zonage_type: <DetailsFricheByZonageType />,
75+
};
76+
7277
export const FrichesCharts: React.FC = () => {
7378
const { landId, landType, showCharts } = useFrichesContext();
7479

@@ -86,13 +91,7 @@ export const FrichesCharts: React.FC = () => {
8691
sources={chart.sources}
8792
showDataTable={true}
8893
>
89-
{chart.id === "friche_zonage_environnemental" && (
90-
<DetailsFricheZonageEnvironnemental />
91-
)}
92-
{chart.id === "friche_surface" && <DetailsFricheBySize />}
93-
{chart.id === "friche_zonage_type" && (
94-
<DetailsFricheByZonageType />
95-
)}
94+
{FRICHE_DETAILS[chart.id]}
9695
</FrichesChart>
9796
</BaseCard>
9897
</div>

0 commit comments

Comments
 (0)