From 41244e5354fd05ef92d94fda0ceab36b2c23e88d Mon Sep 17 00:00:00 2001
From: Anton Tananaev <anton@traccar.org>
Date: Sat, 14 May 2022 16:58:54 -0700
Subject: Improve report filter

---
 modern/src/reports/ChartReportPage.js | 57 +++++++++++++----------------------
 1 file changed, 21 insertions(+), 36 deletions(-)

(limited to 'modern/src/reports/ChartReportPage.js')

diff --git a/modern/src/reports/ChartReportPage.js b/modern/src/reports/ChartReportPage.js
index 4ddb5f75..57c7c689 100644
--- a/modern/src/reports/ChartReportPage.js
+++ b/modern/src/reports/ChartReportPage.js
@@ -1,8 +1,8 @@
 import React, { useState } from 'react';
 import {
-  Grid, FormControl, InputLabel, Select, MenuItem,
+  FormControl, InputLabel, Select, MenuItem,
 } from '@material-ui/core';
-import ReportFilter from './components/ReportFilter';
+import ReportFilter, { useFilterStyles } from './components/ReportFilter';
 import Graph from './components/Graph';
 import { useAttributePreference } from '../common/util/preferences';
 import { formatDate } from '../common/util/formatter';
@@ -11,9 +11,15 @@ import { useTranslation } from '../common/components/LocalizationProvider';
 import PageLayout from '../common/components/PageLayout';
 import ReportsMenu from './components/ReportsMenu';
 
-const Filter = ({ children, setItems }) => {
+const ChartReportPage = () => {
+  const classes = useFilterStyles();
+  const t = useTranslation();
+
   const speedUnit = useAttributePreference('speedUnit');
 
+  const [items, setItems] = useState([]);
+  const [type, setType] = useState('speed');
+
   const handleSubmit = async (deviceId, from, to, mail, headers) => {
     const query = new URLSearchParams({
       deviceId, from, to, mail,
@@ -30,42 +36,21 @@ const Filter = ({ children, setItems }) => {
       setItems(formattedPositions);
     }
   };
-  return (
-    <>
-      <ReportFilter handleSubmit={handleSubmit} showOnly />
-      {children}
-    </>
-  );
-};
-
-const ChartType = ({ type, setType }) => {
-  const t = useTranslation();
-
-  return (
-    <Grid container spacing={3}>
-      <Grid item xs={12} sm={6}>
-        <FormControl variant="filled" margin="normal" fullWidth>
-          <InputLabel>{t('reportChartType')}</InputLabel>
-          <Select value={type} onChange={(e) => setType(e.target.value)}>
-            <MenuItem value="speed">{t('positionSpeed')}</MenuItem>
-            <MenuItem value="accuracy">{t('positionAccuracy')}</MenuItem>
-            <MenuItem value="altitude">{t('positionAltitude')}</MenuItem>
-          </Select>
-        </FormControl>
-      </Grid>
-    </Grid>
-  );
-};
-
-const ChartReportPage = () => {
-  const [items, setItems] = useState([]);
-  const [type, setType] = useState('speed');
 
   return (
     <PageLayout menu={<ReportsMenu />} breadcrumbs={['reportTitle', 'reportChart']}>
-      <Filter setItems={setItems}>
-        <ChartType type={type} setType={setType} />
-      </Filter>
+      <ReportFilter handleSubmit={handleSubmit} showOnly>
+        <div className={classes.item}>
+          <FormControl variant="filled" fullWidth>
+            <InputLabel>{t('reportChartType')}</InputLabel>
+            <Select value={type} onChange={(e) => setType(e.target.value)}>
+              <MenuItem value="speed">{t('positionSpeed')}</MenuItem>
+              <MenuItem value="accuracy">{t('positionAccuracy')}</MenuItem>
+              <MenuItem value="altitude">{t('positionAltitude')}</MenuItem>
+            </Select>
+          </FormControl>
+        </div>
+      </ReportFilter>
       <Graph items={items} type={type} />
     </PageLayout>
   );
-- 
cgit v1.2.3