Update click_outside to be more robust

packages/bbui/src/Actions/click_outside.js

@@ -1,8 +1,13 @@
 const ignoredClasses = [
-  ".flatpickr-calendar",
-  ".spectrum-Calendar",
-  ".spectrum-Popover",
   ".download-js-link",
+  ".flatpickr-calendar",
+  ".spectrum-Menu",
+  ".date-time-popover",
+]
+const conditionallyIgnoredClasses = [
+  ".spectrum-Underlay",
+  ".drawer-wrapper",
+  ".spectrum-Popover",
 ]
 let clickHandlers = []
 
@@ -22,26 +27,23 @@ const handleClick = event => {
 
   // Process handlers
   clickHandlers.forEach(handler => {
+    // Check that we're the right kind of click event
+    if (handler.allowedType && event.type !== handler.allowedType) {
+      return
+    }
+
+    // Check that the click isn't inside the target
     if (handler.element.contains(event.target)) {
       return
     }
 
-    // Ignore clicks for modals, unless the handler is registered from a modal
+    // Ignore clicks for certain classes unless we're nested inside them
-    const sourceInModal = handler.anchor.closest(".spectrum-Underlay") != null
+    for (let className of conditionallyIgnoredClasses) {
-    const clickInModal = event.target.closest(".spectrum-Underlay") != null
+      const sourceInside = handler.anchor.closest(className) != null
-    if (clickInModal && !sourceInModal) {
+      const clickInside = event.target.closest(className) != null
-      return
+      if (clickInside && !sourceInside) {
-    }
+        return
-
+      }
-    // Ignore clicks for drawers, unless the handler is registered from a drawer
-    const sourceInDrawer = handler.anchor.closest(".drawer-wrapper") != null
-    const clickInDrawer = event.target.closest(".drawer-wrapper") != null
-    if (clickInDrawer && !sourceInDrawer) {
-      return
-    }
-
-    if (handler.allowedType && event.type !== handler.allowedType) {
-      return
     }
 
     handler.callback?.(event)