clean up cursor pointers (onyx-dot-app#3385)

* update * nit
aronszanto · Dec 23, 2024 · cdef685 · cdef685
1 parent 3d0a199
commit cdef685
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 25 deletions.
diff --git a/web/src/components/admin/ClientLayout.tsx b/web/src/components/admin/ClientLayout.tsx
@@ -405,7 +405,7 @@ export function ClientLayout({
           />
         </div>
         <div className="pb-8 relative h-full overflow-y-auto w-full">
-          <div className="fixed left-0 gap-x-4 px-4 top-2 h-8 px-2 w-full items-center flex justify-end">
+          <div className="fixed left-0 gap-x-4 px-2 top-2 h-8 px-0 mb-auto w-full items-start flex justify-end">
             <UserDropdown />
           </div>
           <div className="pt-20 flex overflow-y-auto overflow-x-hidden h-full px-4 md:px-12">

diff --git a/web/src/components/chat_search/Header.tsx b/web/src/components/chat_search/Header.tsx
@@ -75,7 +75,7 @@ export default function FunctionalHeader({
   };
   return (
     <div className="left-0 sticky top-0 z-20 w-full relative flex">
-      <div className="items-end flex  cursor-pointer text-text-700 relative flex w-full">
+      <div className="items-end flex mt-2 cursor-pointer text-text-700 relative flex w-full">
         <LogoType
           assistantId={currentChatSession?.persona_id}
           page={page}
@@ -110,13 +110,13 @@ export default function FunctionalHeader({
             />
           </div>
 
-          <div className="absolute right-0 top-2 h-8 items-end flex gap-x-2">
+          <div className="absolute right-0 top-0 flex gap-x-2">
             {setSharingModalVisible && (
               <div
                 onClick={() => setSharingModalVisible(true)}
-                className="mobile:hidden p-1 my-auto rounded cursor-pointer hover:bg-hover-light"
+                className="mobile:hidden my-auto rounded cursor-pointer hover:bg-hover-light"
               >
-                <FiShare2 size="16" />
+                <FiShare2 size="18" />
               </div>
             )}
             <div className="mobile:hidden flex my-auto">
@@ -139,20 +139,6 @@ export default function FunctionalHeader({
                 <NewChatIcon size={20} />
               </div>
             </Link>
-            <div
-              style={{ transition: "width 0.30s ease-out" }}
-              className={`
-              hidden
-              md:flex 
-              mx-auto
-              overflow-y-hidden 
-              transition-all 
-              duration-300 
-              ease-in-out
-              h-full
-              ${documentSidebarToggled ? "w-[400px]" : "w-[0px]"}
-              `}
-            />
           </div>
           <div
             style={{ transition: "width 0.30s ease-out" }}

diff --git a/web/src/components/header/LogoType.tsx b/web/src/components/header/LogoType.tsx
@@ -44,7 +44,7 @@ export default function LogoType({
     <div
       className={`${
         hideOnMobile && "mobile:hidden"
-      } z-[100] mt-2 h-8 mb-auto shrink-0 flex items-center text-xl`}
+      } z-[100] mt-2 h-8 mb-auto shrink-0 flex  items-center text-xl`}
     >
       {toggleSidebar && page == "chat" ? (
         <button
@@ -62,7 +62,7 @@ export default function LogoType({
         </div>
       )}
       <div
-        className={`cursor-pointer ${
+        className={`${
           showArrow ? "desktop:invisible" : "invisible"
         } break-words inline-block w-fit ml-2 text-text-700 text-xl`}
       >
@@ -87,7 +87,7 @@ export default function LogoType({
           <Tooltip>
             <TooltipTrigger asChild>
               <Link
-                className="my-auto pt-1 mobile:hidden"
+                className="mb-auto mobile:hidden"
                 href={
                   `/${page}` +
                   (NEXT_PUBLIC_NEW_CHAT_DIRECTS_TO_SAME_PERSONA && assistantId
@@ -103,9 +103,10 @@ export default function LogoType({
                   }
                 }}
               >
-                <div className="cursor-pointer ml-2 my-auto  text-text-700 hover:text-text-600 transition-colors duration-300">
-                  <NewChatIcon size={20} />
-                </div>
+                <NewChatIcon
+                  className="ml-2 flex-none text-text-700 hover:text-text-600 transition-colors duration-300"
+                  size={20}
+                />
               </Link>
             </TooltipTrigger>
             <TooltipContent>New Chat</TooltipContent>