From e91aa652e7dcf8dd0911e27b805ca9d96fa6a854 Mon Sep 17 00:00:00 2001
From: Eric Doughty-Papassideris <edoughtypapassideris@linagora.com>
Date: Thu, 10 Oct 2024 22:48:52 +0200
Subject: [PATCH 1/2] =?UTF-8?q?=F0=9F=92=84=20front:=20fix=20file=20browse?=
 =?UTF-8?q?r=20vertical=20borders=20(and=20fix=20react=20warning)=20(#645)?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../src/app/views/client/body/drive/browser.tsx        | 10 +++++-----
 .../views/client/body/drive/documents/document-row.tsx |  4 ++--
 .../views/client/body/drive/documents/folder-row.tsx   |  2 +-
 3 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/tdrive/frontend/src/app/views/client/body/drive/browser.tsx b/tdrive/frontend/src/app/views/client/body/drive/browser.tsx
index ac223b9b..cc555c55 100644
--- a/tdrive/frontend/src/app/views/client/body/drive/browser.tsx
+++ b/tdrive/frontend/src/app/views/client/body/drive/browser.tsx
@@ -227,8 +227,8 @@ export default memo(
       const commonProps = {
         key: index,
         className:
-          (index === 0 ? 'rounded-t-md ' : '') +
-          (index === documents.length - 1 ? 'rounded-b-md ' : ''),
+          (index === 0 ? 'rounded-t-md ' : '-mt-px ') +
+          (index === items.length - 1 ? 'rounded-b-md ' : ''),
         item: child,
         checked: checked[child.id] || false,
         onCheck: (v: boolean) => setChecked(_.pickBy({ ...checked, [child.id]: v }, _.identity)),
@@ -238,7 +238,7 @@ export default memo(
       return isMobile ? (
         <DocumentRow {...commonProps} />
       ) : (
-        <Draggable id={index}>
+        <Draggable id={index} key={index}>
           <DocumentRow {...commonProps} />
         </Draggable>
       );
@@ -450,7 +450,7 @@ export default memo(
                         <FolderRow
                           key={index}
                           className={
-                            (index === 0 ? 'rounded-t-md ' : '') +
+                            (index === 0 ? 'rounded-t-md ' : '-mt-px ') +
                             (index === items.length - 1 ? 'rounded-b-md ' : '')
                           }
                           item={child}
@@ -476,7 +476,7 @@ export default memo(
                     {activeIndex ? (
                       <DocumentRowOverlay
                         className={
-                          (activeIndex === 0 ? 'rounded-t-md ' : '') +
+                          (activeIndex === 0 ? 'rounded-t-md ' : '-mt-px ') +
                           (activeIndex === items.length - 1 ? 'rounded-b-md ' : '')
                         }
                         item={activeChild}
diff --git a/tdrive/frontend/src/app/views/client/body/drive/documents/document-row.tsx b/tdrive/frontend/src/app/views/client/body/drive/documents/document-row.tsx
index 059fded5..e8ad0f88 100644
--- a/tdrive/frontend/src/app/views/client/body/drive/documents/document-row.tsx
+++ b/tdrive/frontend/src/app/views/client/body/drive/documents/document-row.tsx
@@ -39,7 +39,7 @@ export const DocumentRow = ({
   return (
     <div
       className={
-        'flex flex-row items-center border border-zinc-200 dark:border-zinc-800 -mt-px px-4 py-3 cursor-pointer ' +
+        'flex flex-row items-center border border-zinc-200 dark:border-zinc-800 px-4 py-3 cursor-pointer ' +
         (checked
           ? 'bg-blue-500 bg-opacity-10 hover:bg-opacity-25  '
           : 'hover:bg-zinc-500 hover:bg-opacity-10 ') +
@@ -96,7 +96,7 @@ export const DocumentRowOverlay = ({ item, className }: DriveItemOverlayProps) =
   return (
     <div
       className={
-        'flex flex-row items-center border border-zinc-200 dark:border-zinc-800 -mt-px px-4 py-3 cursor-pointer ' +
+        'flex flex-row items-center border border-zinc-200 dark:border-zinc-800 px-4 py-3 cursor-pointer ' +
         (className || '')
       }
     >
diff --git a/tdrive/frontend/src/app/views/client/body/drive/documents/folder-row.tsx b/tdrive/frontend/src/app/views/client/body/drive/documents/folder-row.tsx
index a5609df3..d50f6185 100644
--- a/tdrive/frontend/src/app/views/client/body/drive/documents/folder-row.tsx
+++ b/tdrive/frontend/src/app/views/client/body/drive/documents/folder-row.tsx
@@ -23,7 +23,7 @@ export const FolderRow = ({
   return (
     <div
       className={
-        'flex flex-row items-center border border-zinc-200 dark:border-zinc-800 -mt-px px-4 py-3 cursor-pointer ' +
+        'flex flex-row items-center border border-zinc-200 dark:border-zinc-800 px-4 py-3 cursor-pointer ' +
         (className || '') +
         (checked
           ? 'bg-blue-500 bg-opacity-10 hover:bg-opacity-25 '

From d6934c4d06e1de7d98c6a57859436b983882e210 Mon Sep 17 00:00:00 2001
From: Eric Doughty-Papassideris <edoughtypapassideris@linagora.com>
Date: Thu, 10 Oct 2024 23:06:20 +0200
Subject: [PATCH 2/2] =?UTF-8?q?=F0=9F=92=84=20front:=20User=20rights=20mod?=
 =?UTF-8?q?al=20had=20similar=20border=20issues=20as=20browser=20(#645)?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../drive/modals/update-access/internal-users-access.tsx   | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/tdrive/frontend/src/app/views/client/body/drive/modals/update-access/internal-users-access.tsx b/tdrive/frontend/src/app/views/client/body/drive/modals/update-access/internal-users-access.tsx
index fc7e9ea1..043caa7e 100644
--- a/tdrive/frontend/src/app/views/client/body/drive/modals/update-access/internal-users-access.tsx
+++ b/tdrive/frontend/src/app/views/client/body/drive/modals/update-access/internal-users-access.tsx
@@ -81,7 +81,10 @@ export const InternalUsersAccessManager = ({
   return (
     <>
       <div className="rounded-md border-t mt-2 dark:border-zinc-700">
-        <div className="p-4 flex flex-row items-center justify-center rounded-t-md border-x dark:border-zinc-700">
+        <div className={
+          "p-4 flex flex-row items-center justify-center rounded-t-md border-x dark:border-zinc-700"
+          + (showResults ? ' rounded-b-md' : '')
+          }>
           <div className="grow">
             <InputDecorationIcon
               prefix={SearchIcon}
@@ -157,7 +160,7 @@ export const InternalUsersAccessManager = ({
             id={id}
             userId={user?.id}
             disabled={disabled}
-            className={(showResults ? '!rounded-none !rounded-t-md' : '') + (index === usersWithAccess.length - 1 ? ' rounded-b-md border-y' : ' border-t')}
+            className={(showResults ? '!rounded-none' : '') + (index === usersWithAccess.length - 1 ? ' !rounded-b-md border-y' : ' border-t')}
             />
           )}
         {(usersWithAccess?.length ?? 0) < minUserHeight &&