@gitlab/ui 82.0.1 → 82.1.0

package/CHANGELOG.md

@@ -1,3 +1,10 @@
+# [82.1.0](https://gitlab.com/gitlab-org/gitlab-ui/compare/v82.0.1...v82.1.0) (2024-06-20)
+
+
+### Features
+
+* **CSS:** include design token CSS custom properties in gitlab_ui.scss ([1a8db73](https://gitlab.com/gitlab-org/gitlab-ui/commit/1a8db73bde228ef91ee86660515c180984bdeebf))
+
 ## [82.0.1](https://gitlab.com/gitlab-org/gitlab-ui/compare/v82.0.0...v82.0.1) (2024-06-19)
 
 

package/dist/directives/outside/outside.js

@@ -1,7 +1,5 @@
-import { getEventLikeTimeStamp } from './get_event_like_time_stamp';
-
 /**
- * Map<HTMLElement, { bindTimeStamp: number, callback: Function }>
+ * Map<HTMLElement, Function>
  */
 const callbacks = new Map();
 
@@ -11,17 +9,11 @@ const callbacks = new Map();
 let listening = false;
 let lastMousedown = null;
 const globalListener = event => {
-  callbacks.forEach((_ref, element) => {
-    let {
-      bindTimeStamp,
-      callback
-    } = _ref;
+  callbacks.forEach((callback, element) => {
     const originalEvent = lastMousedown || event;
     if (
     // Ignore events that aren't targeted outside the element
-    element.contains(originalEvent.target) ||
-    // Only consider events triggered after the directive was bound
-    event.timeStamp <= bindTimeStamp) {
+    element.contains(originalEvent.target)) {
       return;
     }
     try {
@@ -47,6 +39,9 @@ const startListening = () => {
     return;
   }
   document.addEventListener('mousedown', onMousedown);
+  // Added { capture: true } to prevent the behavior discussed in https://gitlab.com/gitlab-org/gitlab-ui/-/merge_requests/1686#note_412545027
+  // Ensures the event listener handles the event in the capturing phase, avoiding issues encountered previously.
+  // Cannot be tested with Jest or Cypress, but can be tested with Playwright in the future: https://gitlab.com/gitlab-org/gitlab-ui/-/merge_requests/4272#note_1947425384
   document.addEventListener('click', globalListener, {
     capture: true
   });
@@ -61,11 +56,11 @@ const stopListening = () => {
   document.removeEventListener('click', globalListener);
   listening = false;
 };
-const bind = (el, _ref2) => {
+const bind = (el, _ref) => {
   let {
     value,
     arg = 'click'
-  } = _ref2;
+  } = _ref;
   if (typeof value !== 'function') {
     throw new Error(`[GlOutsideDirective] Value must be a function; got ${typeof value}!`);
   }
@@ -87,10 +82,7 @@ const bind = (el, _ref2) => {
   if (!listening) {
     startListening();
   }
-  callbacks.set(el, {
-    bindTimeStamp: getEventLikeTimeStamp(),
-    callback: value
-  });
+  callbacks.set(el, value);
 };
 const unbind = el => {
   callbacks.delete(el);