front: remove popover-related redundant code in DatePicker

Signed-off-by: Stanislas Signoud (Signez) <signez@stanisoft.net>

Author: Signez

front/ui/ui-core/src/components/DatePicker/useDatePicker.ts

@@ -1,6 +1,5 @@
 import { useState, useRef, useEffect } from 'react';
 
-import useOutsideClick from '../../hooks/useOutsideClick';
 import type { StatusWithMessage } from '../StatusMessage';
 import type { DatePickerProps } from './DatePicker';
 import {
@@ -23,17 +22,6 @@ export default function useDatePicker(datePickerProps: DatePickerProps) {
   const calendarPickerRef = useRef<HTMLDivElement>(null);
   const inputRef = useRef<HTMLInputElement>(null);
 
-  useOutsideClick(calendarPickerRef, (e) => {
-    // Do not close the picker if any children in input wrapper is clicked.
-    // This wrapper include, the input itself, the trailing content (which contains the calendar icon) and the leading content
-    if (
-      inputRef.current &&
-      inputRef.current.parentElement?.parentElement?.contains(e.target as Node)
-    )
-      return;
-    setShowPicker(false);
-  });
-
   const handleInputClick = () => {
     if (isRangeMode) {
       setShowPicker(true);

front/ui/ui-core/src/components/InputPopover.tsx

@@ -34,7 +34,7 @@ const InputPopover = ({
       popoverRef.current.showPopover();
       calculatePosition();
     }
-  }, [isOpen, modalRef, calculatePosition]);
+  }, [isOpen, calculatePosition]);
 
   useOutsideClick(popoverRef, onClose);
 

front/ui/ui-core/src/hooks/usePopoverPosition.tsx

@@ -32,7 +32,6 @@ export const usePopoverPosition = (
   }, [inputRef, popoverRef, verticalOffset]);
 
   useEffect(() => {
-    calculatePosition();
     const handleResize = () => calculatePosition();
     window.addEventListener('resize', handleResize);
     return () => window.removeEventListener('resize', handleResize);