From c00b4eea6f2effdc3a61b965005acf411a7e2744 Mon Sep 17 00:00:00 2001
From: Chris Wilton-Magras <chriswilty@zoho.com>
Date: Thu, 28 Mar 2024 16:53:05 +0000
Subject: [PATCH] Remove redundant docviewer button component (#885)

---
 .../components/ControlPanel/ControlPanel.css    |  5 +++++
 .../components/ControlPanel/ControlPanel.tsx    |  9 +++++++--
 .../DocumentViewer/DocumentViewButton.css       | 15 ---------------
 .../DocumentViewer/DocumentViewButton.tsx       | 17 -----------------
 4 files changed, 12 insertions(+), 34 deletions(-)
 delete mode 100644 frontend/src/components/DocumentViewer/DocumentViewButton.css
 delete mode 100644 frontend/src/components/DocumentViewer/DocumentViewButton.tsx

diff --git a/frontend/src/components/ControlPanel/ControlPanel.css b/frontend/src/components/ControlPanel/ControlPanel.css
index 59ca2cef0..e4b97987f 100644
--- a/frontend/src/components/ControlPanel/ControlPanel.css
+++ b/frontend/src/components/ControlPanel/ControlPanel.css
@@ -50,3 +50,8 @@
 .control-panel .config-container > .details-content {
 	margin: 0;
 }
+
+.control-panel .document-view-button {
+	width: 100%;
+	height: 2.5rem;
+}
diff --git a/frontend/src/components/ControlPanel/ControlPanel.tsx b/frontend/src/components/ControlPanel/ControlPanel.tsx
index db096b034..f3f038863 100644
--- a/frontend/src/components/ControlPanel/ControlPanel.tsx
+++ b/frontend/src/components/ControlPanel/ControlPanel.tsx
@@ -1,8 +1,8 @@
 import { DEFENCES_HIDDEN_LEVEL3_IDS, MODEL_DEFENCES } from '@src/Defences';
 import DefenceBox from '@src/components/DefenceBox/DefenceBox';
-import DocumentViewButton from '@src/components/DocumentViewer/DocumentViewButton';
 import ModelBox from '@src/components/ModelBox/ModelBox';
 import DetailElement from '@src/components/ThemedButtons/DetailElement';
+import ThemedButton from '@src/components/ThemedButtons/ThemedButton';
 import { ChatMessage, ChatModel } from '@src/models/chat';
 import {
 	DEFENCE_ID,
@@ -112,7 +112,12 @@ function ControlPanel({
 
 			{/* only show document viewer button in sandbox mode */}
 			{currentLevel === LEVEL_NAMES.SANDBOX && (
-				<DocumentViewButton openDocumentViewer={openDocumentViewer} />
+				<ThemedButton
+					className="document-view-button"
+					onClick={openDocumentViewer}
+				>
+					View Documents
+				</ThemedButton>
 			)}
 		</div>
 	);
diff --git a/frontend/src/components/DocumentViewer/DocumentViewButton.css b/frontend/src/components/DocumentViewer/DocumentViewButton.css
deleted file mode 100644
index ead7b2dc6..000000000
--- a/frontend/src/components/DocumentViewer/DocumentViewButton.css
+++ /dev/null
@@ -1,15 +0,0 @@
-.document-view-button-wrapper {
-	display: flex;
-	flex-shrink: 0;
-	height: 2.5rem;
-}
-
-/* Button fills all available width */
-.document-view-button-wrapper .themed-button-wrapper {
-	display: flex;
-	flex-grow: 1;
-}
-
-.document-view-button-wrapper .themed-button-wrapper button {
-	flex-grow: 1;
-}
diff --git a/frontend/src/components/DocumentViewer/DocumentViewButton.tsx b/frontend/src/components/DocumentViewer/DocumentViewButton.tsx
deleted file mode 100644
index 27a377a39..000000000
--- a/frontend/src/components/DocumentViewer/DocumentViewButton.tsx
+++ /dev/null
@@ -1,17 +0,0 @@
-import ThemedButton from '@src/components/ThemedButtons/ThemedButton';
-
-import './DocumentViewButton.css';
-
-function DocumentViewButton({
-	openDocumentViewer,
-}: {
-	openDocumentViewer: () => void;
-}) {
-	return (
-		<div className="document-view-button-wrapper">
-			<ThemedButton onClick={openDocumentViewer}>View Documents</ThemedButton>
-		</div>
-	);
-}
-
-export default DocumentViewButton;