feat(Popover): prevent closing by click in case of using openOnHover …

…property (#1085)
gravity-ui · Nov 3, 2023 · 2c17a06 · 2c17a06
1 parent 55885d0
commit 2c17a06
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/src/components/Popover/Popover.tsx b/src/components/Popover/Popover.tsx
@@ -207,6 +207,7 @@ export const Popover = React.forwardRef<PopoverInstanceProps, PopoverProps & QAP
                 closeTooltip={closeTooltip}
                 openTooltip={openTooltip}
                 open={isOpen}
+                openOnHover={openOnHover}
                 className={cnPopover('handler')}
                 disabled={disabled}
                 onClick={onClick}

diff --git a/src/components/Popover/components/Trigger/Trigger.tsx b/src/components/Popover/components/Trigger/Trigger.tsx
@@ -12,6 +12,7 @@ export interface TriggerProps {
      * Tooltip's opened state
      */
     open: boolean;
+    openOnHover?: boolean;
     /**
      * Css class for the control
      */
@@ -44,6 +45,7 @@ export interface TriggerProps {
 
 export const Trigger = ({
     open,
+    openOnHover,
     disabled,
     className,
     openTooltip,
@@ -53,7 +55,11 @@ export const Trigger = ({
     children,
 }: TriggerProps) => {
     const handleClick = async (event: React.MouseEvent<HTMLDivElement>) => {
-        if (disabled) {
+        // Ignores click that should close tooltip in case of {openOnHover: true}
+        // to prevent situation when user could close tooltip accidentally
+        const shouldPreventClosingByClick = open && openOnHover;
+
+        if (disabled || shouldPreventClosingByClick) {
             return;
         }