From b8b25baf5c645afd8e0fd93f5fa649059c2759a8 Mon Sep 17 00:00:00 2001
From: SujalXplores <shahc9437@gmail.com>
Date: Mon, 25 Nov 2024 20:46:17 +0530
Subject: [PATCH] refactor: improve history item hover states and interactions

---
 app/components/sidebar/HistoryItem.tsx | 93 +++++++++-----------------
 1 file changed, 30 insertions(+), 63 deletions(-)

diff --git a/app/components/sidebar/HistoryItem.tsx b/app/components/sidebar/HistoryItem.tsx
index 3f58735f8..4c284359d 100644
--- a/app/components/sidebar/HistoryItem.tsx
+++ b/app/components/sidebar/HistoryItem.tsx
@@ -1,5 +1,4 @@
 import * as Dialog from '@radix-ui/react-dialog';
-import { useEffect, useRef, useState } from 'react';
 import { type ChatHistoryItem } from '~/lib/persistence';
 import WithTooltip from '~/components/ui/Tooltip';
 
@@ -11,78 +10,46 @@ interface HistoryItemProps {
 }
 
 export function HistoryItem({ item, onDelete, onDuplicate, exportChat }: HistoryItemProps) {
-  const [hovering, setHovering] = useState(false);
-  const hoverRef = useRef<HTMLDivElement>(null);
-
-  useEffect(() => {
-    let timeout: NodeJS.Timeout | undefined;
-
-    function mouseEnter() {
-      setHovering(true);
-
-      if (timeout) {
-        clearTimeout(timeout);
-      }
-    }
-
-    function mouseLeave() {
-      setHovering(false);
-    }
-
-    hoverRef.current?.addEventListener('mouseenter', mouseEnter);
-    hoverRef.current?.addEventListener('mouseleave', mouseLeave);
-
-    return () => {
-      hoverRef.current?.removeEventListener('mouseenter', mouseEnter);
-      hoverRef.current?.removeEventListener('mouseleave', mouseLeave);
-    };
-  }, []);
-
   return (
-    <div
-      ref={hoverRef}
-      className="group rounded-md text-bolt-elements-textSecondary hover:text-bolt-elements-textPrimary hover:bg-bolt-elements-background-depth-3 overflow-hidden flex justify-between items-center px-2 py-1"
-    >
+    <div className="group rounded-md text-bolt-elements-textSecondary hover:text-bolt-elements-textPrimary hover:bg-bolt-elements-background-depth-3 overflow-hidden flex justify-between items-center px-2 py-1">
       <a href={`/chat/${item.urlId}`} className="flex w-full relative truncate block">
         {item.description}
         <div className="absolute right-0 z-1 top-0 bottom-0 bg-gradient-to-l from-bolt-elements-background-depth-2 group-hover:from-bolt-elements-background-depth-3 box-content pl-3 to-transparent w-10 flex justify-end group-hover:w-15 group-hover:from-99%">
-          {hovering && (
-            <div className="flex items-center p-1 text-bolt-elements-textSecondary">
-              <WithTooltip tooltip="Export chat">
+          <div className="flex items-center p-1 text-bolt-elements-textSecondary opacity-0 group-hover:opacity-100 transition-opacity">
+            <WithTooltip tooltip="Export chat">
+              <button
+                type="button"
+                className="i-ph:download-simple scale-110 mr-2 hover:text-bolt-elements-item-contentAccent"
+                onClick={(event) => {
+                  event.preventDefault();
+                  exportChat(item.id);
+                }}
+                title="Export chat"
+              />
+            </WithTooltip>
+            {onDuplicate && (
+              <WithTooltip tooltip="Duplicate chat">
                 <button
-                  className="i-ph:download-simple scale-110 mr-2"
+                  type="button"
+                  className="i-ph:copy scale-110 mr-2 hover:text-bolt-elements-item-contentAccent"
+                  onClick={() => onDuplicate?.(item.id)}
+                  title="Duplicate chat"
+                />
+              </WithTooltip>
+            )}
+            <Dialog.Trigger asChild>
+              <WithTooltip tooltip="Delete chat">
+                <button
+                  type="button"
+                  className="i-ph:trash scale-110 hover:text-bolt-elements-button-danger-text"
                   onClick={(event) => {
                     event.preventDefault();
-                    exportChat(item.id);
-
-                    //exportChat(item.messages, item.description);
+                    onDelete?.(event);
                   }}
-                  title="Export chat"
                 />
               </WithTooltip>
-              {onDuplicate && (
-                <WithTooltip tooltip="Duplicate chat">
-                  <button
-                    className="i-ph:copy scale-110 mr-2"
-                    onClick={() => onDuplicate?.(item.id)}
-                    title="Duplicate chat"
-                  />
-                </WithTooltip>
-              )}
-              <Dialog.Trigger asChild>
-                <WithTooltip tooltip="Delete chat">
-                  <button
-                    className="i-ph:trash scale-110"
-                    onClick={(event) => {
-                      // we prevent the default so we don't trigger the anchor above
-                      event.preventDefault();
-                      onDelete?.(event);
-                    }}
-                  />
-                </WithTooltip>
-              </Dialog.Trigger>
-            </div>
-          )}
+            </Dialog.Trigger>
+          </div>
         </div>
       </a>
     </div>