From 8cf6db8c65244a1f71956981f1286e43f7dfeca4 Mon Sep 17 00:00:00 2001
From: Charles Bochet <charlesBochet@users.noreply.github.com>
Date: Sun, 13 Aug 2023 02:26:47 +0200
Subject: [PATCH] fix-on-dropdown-menu-width (#1181)

---
 .../modules/ui/dropdown/components/DropdownMenu.tsx    |  2 +-
 front/src/modules/ui/table/components/EntityTable.tsx  |  2 ++
 .../modules/ui/tooltip/OverflowingTextWithTooltip.tsx  | 10 ++--------
 3 files changed, 5 insertions(+), 9 deletions(-)

diff --git a/front/src/modules/ui/dropdown/components/DropdownMenu.tsx b/front/src/modules/ui/dropdown/components/DropdownMenu.tsx
index 12e09ff91..6b8958706 100644
--- a/front/src/modules/ui/dropdown/components/DropdownMenu.tsx
+++ b/front/src/modules/ui/dropdown/components/DropdownMenu.tsx
@@ -18,5 +18,5 @@ export const DropdownMenu = styled.div<{
 
   overflow: hidden;
 
-  width: ${({ width }) => width ?? 160}px;
+  width: ${({ width }) => (width && width > 160 ? width : 160)}px;
 `;
diff --git a/front/src/modules/ui/table/components/EntityTable.tsx b/front/src/modules/ui/table/components/EntityTable.tsx
index 66cc7656f..d3a07d393 100644
--- a/front/src/modules/ui/table/components/EntityTable.tsx
+++ b/front/src/modules/ui/table/components/EntityTable.tsx
@@ -28,6 +28,8 @@ const StyledTable = styled.table`
   margin-right: ${({ theme }) => theme.table.horizontalCellMargin};
   table-layout: fixed;
 
+  width: calc(100% - ${({ theme }) => theme.table.horizontalCellMargin} * 2);
+
   th {
     border: 1px solid ${({ theme }) => theme.border.color.light};
     border-collapse: collapse;
diff --git a/front/src/modules/ui/tooltip/OverflowingTextWithTooltip.tsx b/front/src/modules/ui/tooltip/OverflowingTextWithTooltip.tsx
index b00808284..f19b0436b 100644
--- a/front/src/modules/ui/tooltip/OverflowingTextWithTooltip.tsx
+++ b/front/src/modules/ui/tooltip/OverflowingTextWithTooltip.tsx
@@ -47,11 +47,6 @@ export function OverflowingTextWithTooltip({
     event.preventDefault();
   }
 
-  function handleTooltipMouseUp(event: React.MouseEvent<HTMLDivElement>) {
-    event.stopPropagation();
-    event.preventDefault();
-  }
-
   return (
     <>
       <StyledOverflowingText
@@ -64,12 +59,11 @@ export function OverflowingTextWithTooltip({
       </StyledOverflowingText>
       {isTitleOverflowing &&
         createPortal(
-          <div onMouseUp={handleTooltipMouseUp} onClick={handleTooltipClick}>
+          <div onClick={handleTooltipClick}>
             <AppTooltip
               anchorSelect={`#${textElementId}`}
               content={text ?? ''}
-              clickable
-              delayHide={100}
+              delayHide={0}
               offset={5}
               noArrow
               place="bottom"