@jetbrains/ring-ui 7.0.80 → 7.0.81

package/components/dropdown/dropdown.css

@@ -4,6 +4,10 @@
   display: inline-block;
 }
 
+.clickInterceptor {
+  display: contents;
+}
+
 .anchor.anchor {
   margin-inline: -3px;
   padding-inline: 3px;

package/components/dropdown/dropdown.js

@@ -125,10 +125,12 @@ export default class Dropdown extends Component {
             onContextMenu: hoverMode ? this.handlePopupInteraction : undefined,
             dontCloseOnAnchorClick: true,
         };
-        return (<div data-test={dataTests('ring-dropdown', dataTest)} {...restProps} onClick={clickMode ? this.onClick : undefined} 
-        // anchorElement should be a `button` or an `a`
-        role='presentation' onMouseEnter={hoverMode ? this.onMouseEnter : undefined} onMouseLeave={hoverMode ? this.onMouseLeave : undefined} className={classes}>
-        {anchorElement}
+        return (<div data-test={dataTests('ring-dropdown', dataTest)} {...restProps} onMouseEnter={hoverMode ? this.onMouseEnter : undefined} onMouseLeave={hoverMode ? this.onMouseLeave : undefined} className={classes}>
+        {clickMode ? (<div data-test='ring-dropdown-anchor-click-interceptor' className={styles.clickInterceptor} onClick={this.onClick} 
+            // anchorElement should be a `button` or an `a`
+            role='presentation'>
+            {anchorElement}
+          </div>) : (anchorElement)}
         {typeof children === 'function'
                 ? children(childProps)
                 : cloneElement(children, childProps)}

package/components/popup/popup.js

@@ -20,28 +20,6 @@ import { setCSSAnchorPositioning, supportsCSSAnchorPositioning } from './positio
 import { ThemeContext, WithThemeClasses } from '../global/theme';
 import styles from './popup.css';
 export { PopupTargetContext, PopupTarget };
-const isPossibleClientSideNavigation = (event) => {
-    const target = event.target;
-    const link = target.closest('a');
-    // Taken from https://github.com/nanostores/router/blob/80a333db4cf0789fda21a02715ebabca15192642/index.js#L58-L69
-    return (link &&
-        event.button === 0 && // Left mouse button
-        link.target !== '_blank' && // Not for new tab
-        link.origin === location.origin && // Not external link
-        link.rel !== 'external' && // Not external link
-        link.target !== '_self' && // Now manually disabled
-        !link.download && // Not download link
-        !event.altKey && // Not download link by user
-        !event.metaKey && // Not open in new tab by user
-        !event.ctrlKey && // Not open in new tab by user
-        !event.shiftKey && // Not open in new window by user
-        !event.defaultPrevented);
-};
-const stop = (event) => {
-    if (!isPossibleClientSideNavigation(event)) {
-        event.stopPropagation();
-    }
-};
 export const getPopupContainer = (target) => typeof target === 'string' ? document.querySelector(`[data-portaltarget=${target}]`) : target;
 /**
  * @constructor
@@ -315,11 +293,7 @@ export default class Popup extends PureComponent {
                             [styles.showing]: showing,
                             [styles.largeBorderRadius]: largeBorderRadius,
                         });
-                        return (<span 
-                        // prevent bubbling through portal
-                        onClick={stop} 
-                        // This handler only blocks bubbling through React portal
-                        role='presentation' ref={this.portalRef}>
+                        return (<span ref={this.portalRef}>
                       {this.shouldUseShortcuts() && <Shortcuts map={this.shortcutsMap} scope={this.shortcutsScope}/>}
 
                       {client !== false &&

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jetbrains/ring-ui",
-  "version": "7.0.80",
+  "version": "7.0.81",
   "description": "JetBrains UI library",
   "author": {
     "name": "JetBrains"