Merge pull request #14743 from craftcms/bugfix/14736-click-and-dblclick

when clicking on A or BUTTON prevent dblclick

Fixes #14736

CHANGELOG.md

@@ -14,6 +14,7 @@
 - Fixed a styling issue when editing an entry without any meta fields. ([#14721](https://github.com/craftcms/cms/issues/14721))
 - Fixed a bug where the `_includes/nav.twig` template wasn’t marking nested nav items as selected. ([#14735](https://github.com/craftcms/cms/pull/14735))
 - Fixed issues with menu options’ hover styles.
+- Fixed a bug where double-clicking on an element’s linked label or action button would cause its slideout to open, in addition to the link/button being activated. ([#14736](https://github.com/craftcms/cms/issues/14736))
 
 ## 5.0.0 - 2024-03-26
 

src/web/assets/cp/src/js/BaseElementIndexView.js

@@ -83,8 +83,8 @@ Craft.BaseElementIndexView = Garnish.Base.extend(
       // Enable inline element editing if this is an index page
       if (this.elementIndex.isAdministrative) {
         this._handleElementEditing = (ev) => {
-          if (['A', 'BUTTON'].includes(ev.target.nodeName)) {
-            // Let the link do its thing
+          if ($(ev.target).closest('a,button').length) {
+            // Let the link/button do its thing
             return;
           }
 

src/web/assets/cp/src/js/NestedElementManager.js

@@ -382,7 +382,7 @@ Craft.NestedElementManager = Garnish.Base.extend(
     initElement($element) {
       if (Garnish.hasAttr($element, 'data-editable')) {
         this.addListener($element, 'dblclick,taphold', (ev) => {
-          if (['A', 'BUTTON'].includes(ev.target.nodeName)) {
+          if ($(ev.target).closest('a,button').length) {
             // Let the link/button do its thing
             return;
           }