From 004d98e0d92a5d12917b538a8bfd18837009f72c Mon Sep 17 00:00:00 2001
From: KoalaSat <yv1vtrul@duck.com>
Date: Sun, 15 Oct 2023 23:48:05 +0200
Subject: [PATCH] New icons

---
 frontend/src/components/Dialogs/F2fMap.tsx    | 31 ++++++---
 .../src/components/MakerForm/MakerForm.tsx    |  1 +
 frontend/src/components/Map/index.tsx         | 69 ++++++++++++-------
 3 files changed, 65 insertions(+), 36 deletions(-)

diff --git a/frontend/src/components/Dialogs/F2fMap.tsx b/frontend/src/components/Dialogs/F2fMap.tsx
index 50386ae1..15fb10d3 100644
--- a/frontend/src/components/Dialogs/F2fMap.tsx
+++ b/frontend/src/components/Dialogs/F2fMap.tsx
@@ -95,17 +95,28 @@ const F2fMapDialog = ({
         />
       </DialogContent>
       <DialogActions>
-        <Tooltip
-          enterTouchDelay={0}
-          placement='top'
-          title={t(
-            'To protect your privacy, your selection will be slightly randomized without losing accuracy',
-          )}
-        >
-          <Button color='primary' variant='contained' onClick={onSave} disabled={!position}>
-            {interactive ? t('Save') : t('Close')}
+        {interactive ? (
+          <Tooltip
+            enterTouchDelay={0}
+            placement='top'
+            title={t(
+              'To protect your privacy, your selection will be slightly randomized without losing accuracy',
+            )}
+          >
+            <Button color='primary' variant='contained' onClick={onSave} disabled={!position}>
+              {t('Save')}
+            </Button>
+          </Tooltip>
+        ) : (
+          <Button
+            color='primary'
+            variant='contained'
+            onClick={() => onClose()}
+            disabled={!position}
+          >
+            {t('Close')}
           </Button>
-        </Tooltip>
+        )}
       </DialogActions>
     </Dialog>
   );
diff --git a/frontend/src/components/MakerForm/MakerForm.tsx b/frontend/src/components/MakerForm/MakerForm.tsx
index 02a15d88..4145bc00 100644
--- a/frontend/src/components/MakerForm/MakerForm.tsx
+++ b/frontend/src/components/MakerForm/MakerForm.tsx
@@ -544,6 +544,7 @@ const MakerForm = ({
           if (pos) handleAddLocation(pos);
           setOpenWorldmap(false);
         }}
+        zoom={maker.latitude && maker.longitude ? 6 : undefined}
       />
       <Collapse in={limits.list.length == 0}>
         <div style={{ display: limits.list.length == 0 ? '' : 'none' }}>
diff --git a/frontend/src/components/Map/index.tsx b/frontend/src/components/Map/index.tsx
index cf828631..8dfd624e 100644
--- a/frontend/src/components/Map/index.tsx
+++ b/frontend/src/components/Map/index.tsx
@@ -1,10 +1,18 @@
 import React, { useContext, useEffect, useState } from 'react';
 import { apiClient } from '../../services/api';
-import { MapContainer, GeoJSON, useMapEvents, Circle, TileLayer, Tooltip } from 'react-leaflet';
+import {
+  MapContainer,
+  GeoJSON,
+  useMapEvents,
+  Circle,
+  TileLayer,
+  Tooltip,
+  Marker,
+} from 'react-leaflet';
 import { useTheme, LinearProgress } from '@mui/material';
 import { UseAppStoreType, AppContext } from '../../contexts/AppContext';
 import { GeoJsonObject } from 'geojson';
-import { LeafletMouseEvent } from 'leaflet';
+import { Icon, LeafletMouseEvent, Point } from 'leaflet';
 import { PublicOrder } from '../../models';
 import OrderTooltip from '../Charts/helpers/OrderTooltip';
 
@@ -43,6 +51,37 @@ const Map = ({
     }
   }, []);
 
+  const RobotMarker = (
+    key: string | number,
+    position: [number, number],
+    orderType: number,
+    order?: PublicOrder,
+  ) => {
+    const color = orderType == 1 ? 'Blue' : 'Lilac';
+    return (
+      <Marker
+        key={key}
+        position={position}
+        icon={
+          new Icon({
+            iconUrl: `../../../static/assets/vector/Location_robot_${color}.svg`,
+            iconAnchor: [18, 32],
+            iconSize: [32, 32],
+          })
+        }
+        eventHandlers={{
+          click: (_event: LeafletMouseEvent) => order?.id && onOrderClicked(order.id),
+        }}
+      >
+        {order && (
+          <Tooltip direction='top'>
+            <OrderTooltip order={order} />
+          </Tooltip>
+        )}
+      </Marker>
+    );
+  };
+
   const LocationMarker = () => {
     useMapEvents({
       click(event: LeafletMouseEvent) {
@@ -52,35 +91,13 @@ const Map = ({
       },
     });
 
-    const color = orderType == 1 ? theme.palette.primary.main : theme.palette.secondary.main;
-
-    return position ? (
-      <Circle center={position} pathOptions={{ fillColor: color, color }} radius={10000}></Circle>
-    ) : (
-      <></>
-    );
+    return position ? RobotMarker('marker', position, orderType || 0) : <></>;
   };
 
   const getOrderMarkers = () => {
     return orders.map((order) => {
       if (!order.latitude || !order.longitude) return <></>;
-
-      const color = order.type == 1 ? theme.palette.primary.main : theme.palette.secondary.main;
-      return (
-        <Circle
-          key={order.id}
-          center={[order.latitude, order.longitude]}
-          pathOptions={{ fillColor: color, color }}
-          radius={10000}
-          eventHandlers={{
-            click: (_event: LeafletMouseEvent) => onOrderClicked(order.id),
-          }}
-        >
-          <Tooltip direction='top'>
-            <OrderTooltip order={order} />
-          </Tooltip>
-        </Circle>
-      );
+      return RobotMarker('marker', [order.latitude, order.longitude], orderType || 0, order);
     });
   };