Remove unnecessary js for the button

plotly · davidangarita1 · May 28, 2025 · May 28, 2025 · May 28, 2025 · May 29, 2025
commit 165dffe79a6a0daf7d9c8af727f682de4b6982ca
diff --git a/src/components/modebar/modebar.js b/src/components/modebar/modebar.js
@@ -149,10 +149,9 @@ proto.createButton = function(config) {
     var button = document.createElement('button');
 
     button.setAttribute('type', 'button');
-    button.setAttribute('tabindex', '0');
     button.setAttribute('rel', 'tooltip');
     button.className = 'modebar-btn';
-    
+
     var title = config.title;
     if(title === undefined) title = config.name;
     // for localization: allow title to be a callable that takes gd as arg
@@ -181,15 +180,6 @@ proto.createButton = function(config) {
             // only needed for 'hoverClosestGeo' which does not call relayout
             _this.updateActiveButton(ev.currentTarget);
         });
-        button.addEventListener('keydown', function(e) {
-            if (e.key === 'Enter' || e.key === ' ') {
-                const activeButton = e.target.closest('.modebar-btn');
-                if (activeButton) {
-                    activeButton.click();
-                    e.preventDefault();
-                }
-            }
-        });
     }
 
     button.setAttribute('data-toggle', config.toggle || false);