From 261736bde5d4e2e205db56ae21efdbe45a71591b Mon Sep 17 00:00:00 2001
From: Mathew Pareles <mathewpareles@gmail.com>
Date: Mon, 13 Jan 2025 03:23:38 -0800
Subject: [PATCH] prepare edit button

---
 .../react/src/sidebar-tsx/SidebarChat.tsx     | 74 +++++++++++++++----
 .../src/sidebar-tsx/SidebarThreadSelector.tsx | 27 ++++---
 2 files changed, 73 insertions(+), 28 deletions(-)

diff --git a/src/vs/workbench/contrib/void/browser/react/src/sidebar-tsx/SidebarChat.tsx b/src/vs/workbench/contrib/void/browser/react/src/sidebar-tsx/SidebarChat.tsx
index db450b95..3687938b 100644
--- a/src/vs/workbench/contrib/void/browser/react/src/sidebar-tsx/SidebarChat.tsx
+++ b/src/vs/workbench/contrib/void/browser/react/src/sidebar-tsx/SidebarChat.tsx
@@ -28,6 +28,7 @@ import { useScrollbarStyles } from '../util/useScrollbarStyles.js';
 import { VOID_CTRL_L_ACTION_ID } from '../../../actionIDs.js';
 import { ArrowBigLeftDash, CopyX, Delete, FileX2, SquareX, X } from 'lucide-react';
 import { filenameToVscodeLanguage } from '../../../helpers/detectLanguage.js';
+import { Pencil } from 'lucide-react'
 
 
 export const IconX = ({ size, className = '', ...props }: { size: number, className?: string } & React.SVGProps<SVGSVGElement>) => {
@@ -332,7 +333,7 @@ export const SelectedFiles = (
 								{/* X button */}
 								{type === 'staging' &&
 									<span
-										className='cursor-pointer hover:bg-vscode-toolbar-hover-bg rounded-md z-1'
+										className='cursor-pointer hover:brightness-95 rounded-md z-1'
 										onClick={(e) => {
 											e.stopPropagation(); // don't open/close selection
 											if (type !== 'staging') return;
@@ -346,7 +347,7 @@ export const SelectedFiles = (
 
 							</div>
 
-							{/* delete all selections button */}
+							{/* clear all selections button */}
 							{type !== 'staging' || selections.length === 0 || i !== selections.length - 1
 								? null
 								: <div key={i} className={`flex items-center gap-0.5 ${isThisSelectionOpened ? 'w-full' : ''}`}>
@@ -357,10 +358,14 @@ export const SelectedFiles = (
 									>
 										<Delete
 											size={16}
-											className='stroke-[1] stroke-void-fg-3
-										fill-void-bg-3
-										hover:brightness-105
-										cursor-pointer'
+											className={`stroke-[1]
+												stroke-void-fg-1
+												fill-void-bg-3
+												opacity-40
+												hover:opacity-60
+												transition-all duration-150
+												cursor-pointer
+											`}
 											onClick={() => { setStaging([]) }}
 										/>
 									</div>
@@ -381,7 +386,8 @@ export const SelectedFiles = (
 									maxHeight={100}
 									showScrollbars={true}
 								/>
-							</div>}
+							</div>
+						}
 					</div>
 
 				})}
@@ -393,6 +399,7 @@ export const SelectedFiles = (
 }
 
 
+
 const ChatBubble = ({ chatMessage, isLoading }: {
 	chatMessage: ChatMessage,
 	isLoading?: boolean,
@@ -400,6 +407,9 @@ const ChatBubble = ({ chatMessage, isLoading }: {
 
 	const role = chatMessage.role
 
+	// edit mode state
+	const [isEditMode, setIsEditMode] = useState(false)
+
 	if (!chatMessage.displayContent)
 		return null
 
@@ -409,30 +419,61 @@ const ChatBubble = ({ chatMessage, isLoading }: {
 		chatbubbleContents = <>
 			<SelectedFiles type='past' selections={chatMessage.selections} />
 			{chatMessage.displayContent}
+
+			{/* {!isEditMode ? chatMessage.displayContent : <></>} */}
+			{/* edit mode content */}
+			{/* TODO this should be the same input box as in the Sidebar */}
+			{/* <textarea
+				value={editModeText}
+				className={`
+						w-full max-w-full
+						h-auto min-h-[81px] max-h-[500px]
+						bg-void-bg-1 resize-none
+					`}
+				style={{ marginTop: 0 }}
+				hidden={!isEditMode}
+			/> */}
+
 		</>
 	}
 	else if (role === 'assistant') {
-		chatbubbleContents = <ChatMarkdownRender string={chatMessage.displayContent} /> // sectionsHTML
+		chatbubbleContents = <ChatMarkdownRender string={chatMessage.displayContent} />
 	}
 
 	return <div
 		// align chatbubble accoridng to role
 		className={`
-				${role === 'user' ? `px-2 self-end w-fit max-w-full` : ''}
-				${role === 'assistant' ? `px-2 self-start w-full max-w-full` : ''}
-			`}
+            relative
+			${isEditMode ? 'px-2 w-full max-w-full'
+				: role === 'user' ? `px-2 self-end w-fit max-w-full`
+					: role === 'assistant' ? `px-2 self-start w-full max-w-full` : ''
+			}
+        `}
 	>
 		<div
 			// style chatbubble according to role
 			className={`
-				p-2 text-left space-y-2 rounded-lg
-				overflow-x-auto max-w-full
-				${role === 'user' ? 'bg-vscode-input-bg text-vscode-input-fg' : ''}
-			`}
+                p-2 text-left space-y-2 rounded-lg
+                overflow-x-auto max-w-full
+                ${role === 'user' ? 'bg-void-bg-1 text-void-fg-1' : ''}
+            `}
 		>
 			{chatbubbleContents}
 			{isLoading && <IconLoading className='opacity-50 text-sm' />}
 		</div>
+
+		{/* edit button */}
+		{/* {role === 'user' &&
+			<Pencil
+				size={16}
+				className={`
+					absolute top-0 right-2
+					translate-x-0 -translate-y-0
+					cursor-pointer z-1
+				`}
+				onClick={() => { setIsEditMode(v => !v); }}
+			/>
+		} */}
 	</div>
 }
 
@@ -610,7 +651,7 @@ export const SidebarChat = () => {
 	>
 		{/* thread selector */}
 		<div ref={historyRef}
-			className={`w-full h-auto mb-2 ${isHistoryOpen ? '' : 'hidden'} ring-2 ring-widget-shadow ring-inset z-10`}
+			className={`w-full h-auto ${isHistoryOpen ? '' : 'hidden'} ring-2 ring-widget-shadow ring-inset z-10`}
 		>
 			<SidebarThreadSelector />
 		</div>
@@ -623,6 +664,7 @@ export const SidebarChat = () => {
 				flex flex-col gap-0
 				overflow-x-hidden
 				overflow-y-auto
+				${previousMessages.length > 0 ? 'pt-2' : ''}
 			`}
 			style={{ maxHeight: sidebarDimensions.height - historyDimensions.height - formDimensions.height - 30 }} // the height of the previousMessages is determined by all other heights
 		>
diff --git a/src/vs/workbench/contrib/void/browser/react/src/sidebar-tsx/SidebarThreadSelector.tsx b/src/vs/workbench/contrib/void/browser/react/src/sidebar-tsx/SidebarThreadSelector.tsx
index 3eec9a88..01507164 100644
--- a/src/vs/workbench/contrib/void/browser/react/src/sidebar-tsx/SidebarThreadSelector.tsx
+++ b/src/vs/workbench/contrib/void/browser/react/src/sidebar-tsx/SidebarThreadSelector.tsx
@@ -51,9 +51,8 @@ export const SidebarThreadSelector = () => {
 			</div>
 
 			{/* a list of all the past threads */}
-
-			<div className="px-1 ">
-				<ul className="flex flex-col gap-y-1 overflow-y-auto list-disc">
+			<div className="px-1">
+				<ul className="flex flex-col gap-y-0.5 overflow-y-auto list-disc">
 
 					{sortedThreadIds.length === 0
 
@@ -66,25 +65,26 @@ export const SidebarThreadSelector = () => {
 
 							const pastThread = allThreads[threadId];
 							let firstMsg = null;
-							let secondMsg = null;
+							// let secondMsg = null;
 
 							const firstMsgIdx = pastThread.messages.findIndex(
 								(msg) => msg.role !== 'system' && !!msg.displayContent
 							);
 
 							if (firstMsgIdx !== -1) {
-								firstMsg = truncate(pastThread.messages[firstMsgIdx].displayContent ?? '');
+								// firstMsg = truncate(pastThread.messages[firstMsgIdx].displayContent ?? '');
+								firstMsg = pastThread.messages[firstMsgIdx].displayContent ?? '';
 							} else {
 								firstMsg = '""';
 							}
 
-							const secondMsgIdx = pastThread.messages.findIndex(
-								(msg, i) => msg.role !== 'system' && !!msg.displayContent && i > firstMsgIdx
-							);
+							// const secondMsgIdx = pastThread.messages.findIndex(
+							// 	(msg, i) => msg.role !== 'system' && !!msg.displayContent && i > firstMsgIdx
+							// );
 
-							if (secondMsgIdx !== -1) {
-								secondMsg = truncate(pastThread.messages[secondMsgIdx].displayContent ?? '');
-							}
+							// if (secondMsgIdx !== -1) {
+							// 	secondMsg = truncate(pastThread.messages[secondMsgIdx].displayContent ?? '');
+							// }
 
 							const numMessages = pastThread.messages.filter(
 								(msg) => msg.role !== 'system'
@@ -93,17 +93,20 @@ export const SidebarThreadSelector = () => {
 							return (
 								<li key={pastThread.id}>
 									<button
+									type='button'
 										className={`
 										hover:bg-void-bg-1
 										${threadsState._currentThreadId === pastThread.id ? 'bg-void-bg-1' : ''}
 										rounded-sm px-2 py-1
 										w-full
 										text-left
+										flex items-center
 									`}
 										onClick={() => threadsStateService.switchToThread(pastThread.id)}
 										title={new Date(pastThread.createdAt).toLocaleString()}
 									>
-										{`${firstMsg} (${numMessages})`}
+										<div className='truncate'>{`${firstMsg}`}</div>
+										<div>{`\u00A0(${numMessages})`}</div>
 									</button>
 								</li>
 							);