Merge pull request #61 from code4rena-dev/develop

Bug Fix - Dropdown Updates

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@code4rena/components-library",
-  "version": "4.0.0",
+  "version": "4.0.1",
   "description": "Code4rena's official components library ",
   "types": "./dist/lib.d.ts",
   "exports": {

src/lib/Dropdown/Dropdown.stories.tsx

@@ -24,7 +24,7 @@ SampleComponent.parameters = {
   },
 };
 SampleComponent.args = {
-  id: "",
+  id: "storybook-dropdown",
   wrapperClass: "",
   triggerButtonClass: "",
   openOnHover: true,

src/lib/Dropdown/Dropdown.tsx

@@ -1,4 +1,4 @@
-import React, { useState } from "react";
+import React, { useCallback, useEffect, useState } from "react";
 import clsx from "clsx";
 import { DropdownProps } from "./Dropdown.types";
 import "./Dropdown.scss";
@@ -26,12 +26,35 @@ export const Dropdown = ({
 }: DropdownProps) => {
   const [isOpen, setIsOpen] = useState(false);
 
+
+  const onClickObserver = useCallback((e: Event) => {
+    const dropdownElement = document.querySelector(`#${id} .c4dropdown--listcontainer`);
+    const dropdownTrigger = document.querySelector(`#${id} .c4dropdown--trigger`);
+
+    if (!dropdownElement?.contains(e.target as Node) && !dropdownTrigger?.contains(e.target as Node)) {
+      setIsOpen(false);
+    }
+  }, []);
+
+  useEffect(() => {
+    if (!openOnHover && id != null && id.length > 0) {
+        window.addEventListener('click', onClickObserver);
+    } else {
+      window.removeEventListener('click', onClickObserver);
+    }
+
+    return () => window.removeEventListener('click', onClickObserver);
+  }, [openOnHover]);
+
   return (
     <div
-      id={id ? id : undefined}
+      id={id ?? undefined}
       onMouseEnter={openOnHover ? () => setIsOpen(true) : undefined}
       onMouseLeave={openOnHover ? () => setIsOpen(false) : undefined}
-      className={clsx("c4dropdown", wrapperClass && wrapperClass)}
+      className={clsx(
+        "c4dropdown",
+        wrapperClass && wrapperClass
+      )}
     >
       <button
         onClick={() => setIsOpen(!isOpen)}