fix: do not close slotted tooltip on overlay mousedown (#10259)

Author: web-padawan

packages/tooltip/src/vaadin-tooltip-mixin.js

@@ -553,11 +553,17 @@ export const TooltipMixin = (superClass) =>
     }
 
     /** @private */
-    __onMouseDown() {
+    __onMouseDown(event) {
       if (this.manual) {
         return;
       }
 
+      // Do not close on bubbling mousedown events when
+      // the tooltip is slotted into the target element
+      if (event.composedPath().includes(this)) {
+        return;
+      }
+
       this._stateController.close(true);
     }
 

test/integration/component-tooltip.test.js

@@ -1,6 +1,6 @@
 import { expect } from '@vaadin/chai-plugins';
 import { resetMouse, sendKeys, sendMouseToElement } from '@vaadin/test-runner-commands';
-import { fixtureSync, nextRender, tabKeyDown } from '@vaadin/testing-helpers';
+import { fixtureSync, mousedown, nextRender, tabKeyDown } from '@vaadin/testing-helpers';
 import './not-animated-styles.js';
 import { AccordionPanel } from '@vaadin/accordion/src/vaadin-accordion-panel.js';
 import { Button } from '@vaadin/button/src/vaadin-button.js';
@@ -153,6 +153,15 @@ before(() => {
       }
     });
 
+    it('should not close slotted tooltip on overlay mousedown', () => {
+      mouseenter(tooltip.target);
+
+      expect(tooltipOverlay.opened).to.be.true;
+      const content = tooltip.querySelector('[slot="overlay"]');
+      mousedown(content);
+      expect(tooltipOverlay.opened).to.be.true;
+    });
+
     it('should set has-tooltip attribute on the element', () => {
       expect(element.hasAttribute('has-tooltip')).to.be.true;
     });