Clear browser tabs when cleaning history (#3851)

* Clear browser tabs when cleaning history * Limit history to just 50 Co-authored-by: Curtis David <[email protected]>
MetaMask · Mar 9, 2022 · 3507c90 · 3507c90
1 parent 80302c4
commit 3507c90
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/app/actions/browser/index.js b/app/actions/browser/index.js
@@ -19,6 +19,7 @@ export function addToHistory({ url, name }) {
 export function clearHistory() {
 	return {
 		type: 'CLEAR_BROWSER_HISTORY',
+		id: Date.now(),
 	};
 }
 

diff --git a/app/reducers/browser/index.js b/app/reducers/browser/index.js
@@ -1,3 +1,5 @@
+import AppConstants from '../../core/AppConstants';
+
 const initialState = {
 	history: [],
 	whitelist: [],
@@ -9,7 +11,7 @@ const browserReducer = (state = initialState, action) => {
 		case 'ADD_TO_BROWSER_HISTORY':
 			return {
 				...state,
-				history: [...state.history, { url: action.url, name: action.name }],
+				history: [...state.history, { url: action.url, name: action.name }].slice(0, 50),
 			};
 		case 'ADD_TO_BROWSER_WHITELIST':
 			return {
@@ -20,6 +22,8 @@ const browserReducer = (state = initialState, action) => {
 			return {
 				...state,
 				history: [],
+				tabs: [{ url: AppConstants.HOMEPAGE_URL, id: action.id }],
+				activeTab: action.id,
 			};
 		case 'CLOSE_ALL_TABS':
 			return {