From f86b1bd31edba9434cda373f339d299cc1865c90 Mon Sep 17 00:00:00 2001
From: jjavierdguezas <jjavierdguezas@gmail.com>
Date: Wed, 12 May 2021 19:02:21 +0200
Subject: [PATCH] add Clear Address History button to UserSettings Menu
 resolves #341

---
 desktop-app/app/components/Drawer.js          |  2 +-
 .../app/components/UserPreferences/index.js   | 23 +++++++++++++++++++
 .../services/searchUrlSuggestions/index.js    |  5 +++-
 3 files changed, 28 insertions(+), 2 deletions(-)

diff --git a/desktop-app/app/components/Drawer.js b/desktop-app/app/components/Drawer.js
index 1bdcf875..2748d964 100644
--- a/desktop-app/app/components/Drawer.js
+++ b/desktop-app/app/components/Drawer.js
@@ -65,7 +65,7 @@ const useStyles = makeStyles(theme => ({
     })} 0 rgba(0, 0, 0, 0.75)`,
   },
   drawerContent: {
-    height: '85%',
+    height: '95%',
     overflow: 'auto',
   },
 }));
diff --git a/desktop-app/app/components/UserPreferences/index.js b/desktop-app/app/components/UserPreferences/index.js
index 165c691a..c535bbda 100644
--- a/desktop-app/app/components/UserPreferences/index.js
+++ b/desktop-app/app/components/UserPreferences/index.js
@@ -4,6 +4,7 @@ import cx from 'classnames';
 import FormControlLabel from '@material-ui/core/FormControlLabel';
 import Typography from '@material-ui/core/Typography';
 import Checkbox from '@material-ui/core/Checkbox';
+import Button from '@material-ui/core/Button';
 import TextField from '@material-ui/core/TextField';
 import Input from '@material-ui/core/Input';
 import SettingsIcon from '@material-ui/icons/Settings';
@@ -19,6 +20,7 @@ import {SCREENSHOT_MECHANISM} from '../../constants/values';
 import {notifyPermissionPreferenceChanged} from '../../utils/permissionUtils.js';
 import {PERMISSION_MANAGEMENT_OPTIONS} from '../../constants/permissionsManagement';
 import {setTheme} from '../../actions/browser';
+import {deleteSearchResults} from '../../services/searchUrlSuggestions';
 
 function UserPreference({
   devToolsConfig,
@@ -241,6 +243,27 @@ function UserPreference({
           onScreenShotSaveLocationChange={onChange}
         />
       </div>
+      <div className={commonClasses.sidebarContentSectionContainer}>
+        <div
+          className={cx(
+            commonClasses.flexAlignVerticalMiddle,
+            classes.sectionHeader
+          )}
+        >
+          Adress History
+        </div>
+        <div className={commonClasses.sidebarContentSectionContainer}>
+          <Button
+            variant="contained"
+            color="primary"
+            aria-label="clear address history"
+            component="span"
+            onClick={deleteSearchResults}
+          >
+            Clear Address History
+          </Button>
+        </div>
+      </div>
       <div className={commonClasses.sidebarContentSectionContainer}>
         <div
           className={cx(
diff --git a/desktop-app/app/services/searchUrlSuggestions/index.js b/desktop-app/app/services/searchUrlSuggestions/index.js
index 13afe642..c6fdfb2f 100644
--- a/desktop-app/app/services/searchUrlSuggestions/index.js
+++ b/desktop-app/app/services/searchUrlSuggestions/index.js
@@ -9,7 +9,10 @@ export const getExistingSearchResults = () => settings.get(ADD_SEARCH_RESULTS);
 
 const addUrlToSearchResults = url => settings.set(ADD_SEARCH_RESULTS, url);
 
-const deleteSearchResults = () => settings.delete(ADD_SEARCH_RESULTS);
+export const deleteSearchResults = () => {
+  settings.delete(ADD_SEARCH_RESULTS);
+  previousSearchResults = [];
+};
 
 const _sortedExistingUrlSearchResult = filteredData => {
   // Most visited site should appear first in the list