@atlaskit/editor-common 120.11.0 → 120.11.1

package/CHANGELOG.md

@@ -1,5 +1,26 @@
 # @atlaskit/editor-common
 
+## 120.11.1
+
+### Patch Changes
+
+- [`3ae04490ac0f4`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/3ae04490ac0f4) -
+  Render the disabled mention chip tooltip without React. Behind the
+  `platform_editor_use_vanilla_components` experiment, with no visual change.
+  - Disabled mention chips no longer mount a React tooltip, its portal, and the DOM-to-React event
+    bridge that fed it.
+  - Tooltips release their positioning instance when they close, rather than keeping one alive per
+    hovered chip.
+  - Tooltips remove their own element and attributes when destroyed, so mention and emoji chips no
+    longer leave orphaned popovers behind.
+  - Re-hovering a tooltip while it is closing keeps it open, matching `@atlaskit/tooltip`.
+    Previously the pending hide still landed and the tooltip disappeared under the pointer.
+  - Tooltips close when they hide, instead of waiting on a transition that nothing declares.
+    Required by the change above: releasing the positioning instance while the popover was still
+    open left it behind as an invisible hit target over the content.
+
+- Updated dependencies
+
 ## 120.11.0
 
 ### Minor Changes

package/dist/cjs/monitoring/error.js

@@ -28,7 +28,7 @@ var NETWORK_FAILURE_REGEX = /^network failure/i;
 var RESIZE_OBSERVER_LOOP_REGEX = /ResizeObserver loop completed with undelivered notifications/;
 var SENTRY_DSN = 'https://0b10c8e02fb44d8796c047b102c9bee8@o55978.ingest.sentry.io/4505129224110080';
 var packageName = 'editor-common'; // Sentry doesn't accept '/' in its releases https://docs.sentry.io/platforms/javascript/configuration/releases/
-var packageVersion = "120.10.4";
+var packageVersion = "120.11.0";
 var sanitiseSentryEvents = function sanitiseSentryEvents(data, _hint) {
   // Remove URL as it has UGC
   // Ignored via go/ees007

package/dist/cjs/ui/DropList/index.js

@@ -24,7 +24,7 @@ function _isNativeReflectConstruct() { try { var t = !Boolean.prototype.valueOf.
  * @jsx jsx
  */ // eslint-disable-next-line @atlaskit/ui-styling-standard/use-compiled -- Ignored via go/DSP-18766
 var packageName = "@atlaskit/editor-common";
-var packageVersion = "120.10.4";
+var packageVersion = "120.11.0";
 var halfFocusRing = 1;
 var dropOffset = '0, 8';
 var fadeIn = (0, _react2.keyframes)({

package/dist/cjs/vanilla-tooltip/index.js

@@ -164,14 +164,37 @@ var VanillaTooltip = exports.VanillaTooltip = /*#__PURE__*/function () {
       }));
     }
 
-    // Hide the tooltip if the hide transition has completed
-    this.tooltip.ontransitionend = function () {
-      if (_this.shouldHidePopover) {
-        _this.tooltip.hidePopover();
-      }
-    };
+    // Hide the tooltip if the hide transition has completed.
+    //
+    // Only reachable in the control arm: no consumer declares a transition on the tooltip, so
+    // this never fires today, and the experiment closes the popover in `hide()` regardless.
+    if (!(0, _isExperimentEnabled.isExperimentEnabled)('platform_editor_use_vanilla_components')) {
+      this.tooltip.ontransitionend = function () {
+        if (_this.shouldHidePopover) {
+          _this.tooltip.hidePopover();
+        }
+      };
+    }
   }
+
+  /**
+   * Closes the popover, tolerating it already being closed.
+   *
+   * `hidePopover()` throws `InvalidStateError` when the popover is not showing, and that is a
+   * routine state rather than an exceptional one: showing any other `hint` or `auto` popover
+   * light-dismisses this one without notifying us, so the caller cannot know whether it is
+   * still open.
+   */
   return (0, _createClass2.default)(VanillaTooltip, [{
+    key: "closePopover",
+    value: function closePopover() {
+      try {
+        this.tooltip.hidePopover();
+      } catch (_error) {
+        // Already closed — nothing to do.
+      }
+    }
+  }, {
     key: "createPopperInstance",
     value: function createPopperInstance() {
       this.popperInstance = getCreatePopper()(this.trigger, this.tooltip, {
@@ -192,6 +215,18 @@ var VanillaTooltip = exports.VanillaTooltip = /*#__PURE__*/function () {
       this.listeners.forEach(function (listener) {
         listener();
       });
+
+      // Undoes what the constructor added to the trigger, matching the React `@atlaskit/tooltip`
+      // callers are moving off, whose unmount took its DOM with it.
+      if ((0, _isExperimentEnabled.isExperimentEnabled)('platform_editor_use_vanilla_components')) {
+        this.tooltip.remove();
+        if (this.trigger.getAttribute('aria-describedby') === this.tooltip.id) {
+          this.trigger.removeAttribute('aria-describedby');
+        }
+        if (this.trigger.getAttribute('popovertarget') === this.tooltip.id) {
+          this.trigger.removeAttribute('popovertarget');
+        }
+      }
     }
 
     /**
@@ -243,20 +278,48 @@ var VanillaTooltip = exports.VanillaTooltip = /*#__PURE__*/function () {
       this.shouldHidePopover = true;
       // Disable the event listeners
       this.currentTimeoutId = setTimeout(function () {
-        var _this3$popperInstance;
-        (_this3$popperInstance = _this3.popperInstance) === null || _this3$popperInstance === void 0 || _this3$popperInstance.setOptions(function (options) {
-          return _objectSpread(_objectSpread({}, options), {}, {
-            modifiers: [startingOffset, {
-              name: 'eventListeners',
-              enabled: false
-            }]
+        if ((0, _isExperimentEnabled.isExperimentEnabled)('platform_editor_use_vanilla_components')) {
+          var _this3$popperInstance;
+          // Released rather than reconfigured. The instance was previously kept for the
+          // trigger's whole lifetime, so every element the pointer passed over retained
+          // one — and under `platform-dst-popper-consolidation` an instance owns a
+          // detached React root, so that was a retained root per hovered trigger.
+          //
+          // Destroying is also the only way to release that root's capture-phase
+          // `scroll`/`resize` listeners on the consolidated path, where the
+          // `eventListeners: false` reconfiguration in the control arm is accepted but
+          // does nothing.
+          (_this3$popperInstance = _this3.popperInstance) === null || _this3$popperInstance === void 0 || _this3$popperInstance.destroy();
+          _this3.popperInstance = undefined;
+          _this3.tooltip.style.opacity = '0';
+          _this3.isDisplayed = false;
+
+          // Closed here rather than from `ontransitionend`. No consumer declares a
+          // transition on the tooltip and none sets one inline, so that event never
+          // fires and the popover stayed open in the top layer for good: an invisible
+          // hit target over the content, and — because `showPopover()` throws
+          // `InvalidStateError` on an already-showing popover — an uncaught error out of
+          // the next `mouseenter`, after which the tooltip never appeared again.
+          _this3.closePopover();
+          // Paired with the `visibility` that `show()` sets, so a popover left open by
+          // a failed close cannot intercept pointer events over the content.
+          _this3.tooltip.style.visibility = 'hidden';
+        } else {
+          var _this3$popperInstance2;
+          (_this3$popperInstance2 = _this3.popperInstance) === null || _this3$popperInstance2 === void 0 || _this3$popperInstance2.setOptions(function (options) {
+            return _objectSpread(_objectSpread({}, options), {}, {
+              modifiers: [startingOffset, {
+                name: 'eventListeners',
+                enabled: false
+              }]
+            });
           });
-        });
-        _this3.tooltip.style.opacity = '0';
-        _this3.isDisplayed = false;
-        // If transition animations are disabled immediately hide the popover
-        if (_this3.tooltip.style.transition === 'none') {
-          _this3.tooltip.hidePopover();
+          _this3.tooltip.style.opacity = '0';
+          _this3.isDisplayed = false;
+          // If transition animations are disabled immediately hide the popover
+          if (_this3.tooltip.style.transition === 'none') {
+            _this3.tooltip.hidePopover();
+          }
         }
       }, immediate ? 0 : this.timeout);
     }
@@ -264,18 +327,26 @@ var VanillaTooltip = exports.VanillaTooltip = /*#__PURE__*/function () {
     key: "show",
     value: function show() {
       var _this4 = this;
-      // Bound before the open delay so Escape can dismiss during it, and before the
-      // `isDisplayed` guard so re-hovering a tooltip whose hide is still pending (`hide()`
-      // unbinds synchronously, but the tooltip stays on screen until its timeout) restores
-      // Escape rather than leaving the visible tooltip undismissable.
+      // Ahead of the `isDisplayed` guard, unlike the equivalent below: `hide()` takes effect
+      // synchronously but leaves the tooltip on screen until its timeout, so a pointer
+      // returning in that window finds `isDisplayed` still true and returns early. Cancelling
+      // first keeps the tooltip up and stops it being left undismissable with a hide pending.
       if ((0, _isExperimentEnabled.isExperimentEnabled)('platform_editor_use_vanilla_components')) {
         this.bindEscapeListener();
+        clearTimeout(this.currentTimeoutId);
+        this.shouldHidePopover = false;
       }
       if (this.isDisplayed) {
         return;
       }
-      clearTimeout(this.currentTimeoutId);
-      this.shouldHidePopover = false;
+
+      // Control only: the experiment cancels above, ahead of the guard. Kept gated rather than
+      // left unconditional — despite being a no-op once the block above has run — so that
+      // experiment cleanup can delete it outright instead of having to reason about it.
+      if (!(0, _isExperimentEnabled.isExperimentEnabled)('platform_editor_use_vanilla_components')) {
+        clearTimeout(this.currentTimeoutId);
+        this.shouldHidePopover = false;
+      }
 
       // Make the tooltip visible - but hide until
       this.tooltip.style.visibility = 'hidden';

package/dist/es2019/monitoring/error.js

@@ -14,7 +14,7 @@ const NETWORK_FAILURE_REGEX = /^network failure/i;
 const RESIZE_OBSERVER_LOOP_REGEX = /ResizeObserver loop completed with undelivered notifications/;
 const SENTRY_DSN = 'https://0b10c8e02fb44d8796c047b102c9bee8@o55978.ingest.sentry.io/4505129224110080';
 const packageName = 'editor-common'; // Sentry doesn't accept '/' in its releases https://docs.sentry.io/platforms/javascript/configuration/releases/
-const packageVersion = "120.10.4";
+const packageVersion = "120.11.0";
 const sanitiseSentryEvents = (data, _hint) => {
   // Remove URL as it has UGC
   // Ignored via go/ees007

package/dist/es2019/ui/DropList/index.js

@@ -14,7 +14,7 @@ import withAnalyticsEvents from '@atlaskit/analytics-next/withAnalyticsEvents';
 import { fg } from '@atlaskit/platform-feature-flags/fg';
 import Layer from '../Layer';
 const packageName = "@atlaskit/editor-common";
-const packageVersion = "120.10.4";
+const packageVersion = "120.11.0";
 const halfFocusRing = 1;
 const dropOffset = '0, 8';
 const fadeIn = keyframes({

package/dist/es2019/vanilla-tooltip/index.js

@@ -142,12 +142,33 @@ export class VanillaTooltip {
       }));
     }
 
-    // Hide the tooltip if the hide transition has completed
-    this.tooltip.ontransitionend = () => {
-      if (this.shouldHidePopover) {
-        this.tooltip.hidePopover();
-      }
-    };
+    // Hide the tooltip if the hide transition has completed.
+    //
+    // Only reachable in the control arm: no consumer declares a transition on the tooltip, so
+    // this never fires today, and the experiment closes the popover in `hide()` regardless.
+    if (!isExperimentEnabled('platform_editor_use_vanilla_components')) {
+      this.tooltip.ontransitionend = () => {
+        if (this.shouldHidePopover) {
+          this.tooltip.hidePopover();
+        }
+      };
+    }
+  }
+
+  /**
+   * Closes the popover, tolerating it already being closed.
+   *
+   * `hidePopover()` throws `InvalidStateError` when the popover is not showing, and that is a
+   * routine state rather than an exceptional one: showing any other `hint` or `auto` popover
+   * light-dismisses this one without notifying us, so the caller cannot know whether it is
+   * still open.
+   */
+  closePopover() {
+    try {
+      this.tooltip.hidePopover();
+    } catch (_error) {
+      // Already closed — nothing to do.
+    }
   }
   createPopperInstance() {
     this.popperInstance = getCreatePopper()(this.trigger, this.tooltip, {
@@ -166,6 +187,18 @@ export class VanillaTooltip {
     this.listeners.forEach(listener => {
       listener();
     });
+
+    // Undoes what the constructor added to the trigger, matching the React `@atlaskit/tooltip`
+    // callers are moving off, whose unmount took its DOM with it.
+    if (isExperimentEnabled('platform_editor_use_vanilla_components')) {
+      this.tooltip.remove();
+      if (this.trigger.getAttribute('aria-describedby') === this.tooltip.id) {
+        this.trigger.removeAttribute('aria-describedby');
+      }
+      if (this.trigger.getAttribute('popovertarget') === this.tooltip.id) {
+        this.trigger.removeAttribute('popovertarget');
+      }
+    }
   }
 
   /**
@@ -208,35 +241,71 @@ export class VanillaTooltip {
     this.shouldHidePopover = true;
     // Disable the event listeners
     this.currentTimeoutId = setTimeout(() => {
-      var _this$popperInstance2;
-      (_this$popperInstance2 = this.popperInstance) === null || _this$popperInstance2 === void 0 ? void 0 : _this$popperInstance2.setOptions(options => ({
-        ...options,
-        modifiers: [startingOffset, {
-          name: 'eventListeners',
-          enabled: false
-        }]
-      }));
-      this.tooltip.style.opacity = '0';
-      this.isDisplayed = false;
-      // If transition animations are disabled immediately hide the popover
-      if (this.tooltip.style.transition === 'none') {
-        this.tooltip.hidePopover();
+      if (isExperimentEnabled('platform_editor_use_vanilla_components')) {
+        var _this$popperInstance2;
+        // Released rather than reconfigured. The instance was previously kept for the
+        // trigger's whole lifetime, so every element the pointer passed over retained
+        // one — and under `platform-dst-popper-consolidation` an instance owns a
+        // detached React root, so that was a retained root per hovered trigger.
+        //
+        // Destroying is also the only way to release that root's capture-phase
+        // `scroll`/`resize` listeners on the consolidated path, where the
+        // `eventListeners: false` reconfiguration in the control arm is accepted but
+        // does nothing.
+        (_this$popperInstance2 = this.popperInstance) === null || _this$popperInstance2 === void 0 ? void 0 : _this$popperInstance2.destroy();
+        this.popperInstance = undefined;
+        this.tooltip.style.opacity = '0';
+        this.isDisplayed = false;
+
+        // Closed here rather than from `ontransitionend`. No consumer declares a
+        // transition on the tooltip and none sets one inline, so that event never
+        // fires and the popover stayed open in the top layer for good: an invisible
+        // hit target over the content, and — because `showPopover()` throws
+        // `InvalidStateError` on an already-showing popover — an uncaught error out of
+        // the next `mouseenter`, after which the tooltip never appeared again.
+        this.closePopover();
+        // Paired with the `visibility` that `show()` sets, so a popover left open by
+        // a failed close cannot intercept pointer events over the content.
+        this.tooltip.style.visibility = 'hidden';
+      } else {
+        var _this$popperInstance3;
+        (_this$popperInstance3 = this.popperInstance) === null || _this$popperInstance3 === void 0 ? void 0 : _this$popperInstance3.setOptions(options => ({
+          ...options,
+          modifiers: [startingOffset, {
+            name: 'eventListeners',
+            enabled: false
+          }]
+        }));
+        this.tooltip.style.opacity = '0';
+        this.isDisplayed = false;
+        // If transition animations are disabled immediately hide the popover
+        if (this.tooltip.style.transition === 'none') {
+          this.tooltip.hidePopover();
+        }
       }
     }, immediate ? 0 : this.timeout);
   }
   show() {
-    // Bound before the open delay so Escape can dismiss during it, and before the
-    // `isDisplayed` guard so re-hovering a tooltip whose hide is still pending (`hide()`
-    // unbinds synchronously, but the tooltip stays on screen until its timeout) restores
-    // Escape rather than leaving the visible tooltip undismissable.
+    // Ahead of the `isDisplayed` guard, unlike the equivalent below: `hide()` takes effect
+    // synchronously but leaves the tooltip on screen until its timeout, so a pointer
+    // returning in that window finds `isDisplayed` still true and returns early. Cancelling
+    // first keeps the tooltip up and stops it being left undismissable with a hide pending.
     if (isExperimentEnabled('platform_editor_use_vanilla_components')) {
       this.bindEscapeListener();
+      clearTimeout(this.currentTimeoutId);
+      this.shouldHidePopover = false;
     }
     if (this.isDisplayed) {
       return;
     }
-    clearTimeout(this.currentTimeoutId);
-    this.shouldHidePopover = false;
+
+    // Control only: the experiment cancels above, ahead of the guard. Kept gated rather than
+    // left unconditional — despite being a no-op once the block above has run — so that
+    // experiment cleanup can delete it outright instead of having to reason about it.
+    if (!isExperimentEnabled('platform_editor_use_vanilla_components')) {
+      clearTimeout(this.currentTimeoutId);
+      this.shouldHidePopover = false;
+    }
 
     // Make the tooltip visible - but hide until
     this.tooltip.style.visibility = 'hidden';
@@ -251,10 +320,10 @@ export class VanillaTooltip {
 
     // Enable the event listeners
     this.currentTimeoutId = setTimeout(() => {
-      var _this$popperInstance3, _this$onShow;
+      var _this$popperInstance4, _this$onShow;
       this.tooltip.style.opacity = '1';
       this.tooltip.style.visibility = 'visible';
-      (_this$popperInstance3 = this.popperInstance) === null || _this$popperInstance3 === void 0 ? void 0 : _this$popperInstance3.setOptions(options => ({
+      (_this$popperInstance4 = this.popperInstance) === null || _this$popperInstance4 === void 0 ? void 0 : _this$popperInstance4.setOptions(options => ({
         ...options,
         modifiers: [endingOffset, {
           name: 'eventListeners',

package/dist/esm/monitoring/error.js

@@ -20,7 +20,7 @@ var NETWORK_FAILURE_REGEX = /^network failure/i;
 var RESIZE_OBSERVER_LOOP_REGEX = /ResizeObserver loop completed with undelivered notifications/;
 var SENTRY_DSN = 'https://0b10c8e02fb44d8796c047b102c9bee8@o55978.ingest.sentry.io/4505129224110080';
 var packageName = 'editor-common'; // Sentry doesn't accept '/' in its releases https://docs.sentry.io/platforms/javascript/configuration/releases/
-var packageVersion = "120.10.4";
+var packageVersion = "120.11.0";
 var sanitiseSentryEvents = function sanitiseSentryEvents(data, _hint) {
   // Remove URL as it has UGC
   // Ignored via go/ees007

package/dist/esm/ui/DropList/index.js

@@ -21,7 +21,7 @@ import withAnalyticsEvents from '@atlaskit/analytics-next/withAnalyticsEvents';
 import { fg } from '@atlaskit/platform-feature-flags/fg';
 import Layer from '../Layer';
 var packageName = "@atlaskit/editor-common";
-var packageVersion = "120.10.4";
+var packageVersion = "120.11.0";
 var halfFocusRing = 1;
 var dropOffset = '0, 8';
 var fadeIn = keyframes({

package/dist/esm/vanilla-tooltip/index.js

@@ -157,14 +157,37 @@ export var VanillaTooltip = /*#__PURE__*/function () {
       }));
     }
 
-    // Hide the tooltip if the hide transition has completed
-    this.tooltip.ontransitionend = function () {
-      if (_this.shouldHidePopover) {
-        _this.tooltip.hidePopover();
-      }
-    };
+    // Hide the tooltip if the hide transition has completed.
+    //
+    // Only reachable in the control arm: no consumer declares a transition on the tooltip, so
+    // this never fires today, and the experiment closes the popover in `hide()` regardless.
+    if (!isExperimentEnabled('platform_editor_use_vanilla_components')) {
+      this.tooltip.ontransitionend = function () {
+        if (_this.shouldHidePopover) {
+          _this.tooltip.hidePopover();
+        }
+      };
+    }
   }
+
+  /**
+   * Closes the popover, tolerating it already being closed.
+   *
+   * `hidePopover()` throws `InvalidStateError` when the popover is not showing, and that is a
+   * routine state rather than an exceptional one: showing any other `hint` or `auto` popover
+   * light-dismisses this one without notifying us, so the caller cannot know whether it is
+   * still open.
+   */
   return _createClass(VanillaTooltip, [{
+    key: "closePopover",
+    value: function closePopover() {
+      try {
+        this.tooltip.hidePopover();
+      } catch (_error) {
+        // Already closed — nothing to do.
+      }
+    }
+  }, {
     key: "createPopperInstance",
     value: function createPopperInstance() {
       this.popperInstance = getCreatePopper()(this.trigger, this.tooltip, {
@@ -185,6 +208,18 @@ export var VanillaTooltip = /*#__PURE__*/function () {
       this.listeners.forEach(function (listener) {
         listener();
       });
+
+      // Undoes what the constructor added to the trigger, matching the React `@atlaskit/tooltip`
+      // callers are moving off, whose unmount took its DOM with it.
+      if (isExperimentEnabled('platform_editor_use_vanilla_components')) {
+        this.tooltip.remove();
+        if (this.trigger.getAttribute('aria-describedby') === this.tooltip.id) {
+          this.trigger.removeAttribute('aria-describedby');
+        }
+        if (this.trigger.getAttribute('popovertarget') === this.tooltip.id) {
+          this.trigger.removeAttribute('popovertarget');
+        }
+      }
     }
 
     /**
@@ -236,20 +271,48 @@ export var VanillaTooltip = /*#__PURE__*/function () {
       this.shouldHidePopover = true;
       // Disable the event listeners
       this.currentTimeoutId = setTimeout(function () {
-        var _this3$popperInstance;
-        (_this3$popperInstance = _this3.popperInstance) === null || _this3$popperInstance === void 0 || _this3$popperInstance.setOptions(function (options) {
-          return _objectSpread(_objectSpread({}, options), {}, {
-            modifiers: [startingOffset, {
-              name: 'eventListeners',
-              enabled: false
-            }]
+        if (isExperimentEnabled('platform_editor_use_vanilla_components')) {
+          var _this3$popperInstance;
+          // Released rather than reconfigured. The instance was previously kept for the
+          // trigger's whole lifetime, so every element the pointer passed over retained
+          // one — and under `platform-dst-popper-consolidation` an instance owns a
+          // detached React root, so that was a retained root per hovered trigger.
+          //
+          // Destroying is also the only way to release that root's capture-phase
+          // `scroll`/`resize` listeners on the consolidated path, where the
+          // `eventListeners: false` reconfiguration in the control arm is accepted but
+          // does nothing.
+          (_this3$popperInstance = _this3.popperInstance) === null || _this3$popperInstance === void 0 || _this3$popperInstance.destroy();
+          _this3.popperInstance = undefined;
+          _this3.tooltip.style.opacity = '0';
+          _this3.isDisplayed = false;
+
+          // Closed here rather than from `ontransitionend`. No consumer declares a
+          // transition on the tooltip and none sets one inline, so that event never
+          // fires and the popover stayed open in the top layer for good: an invisible
+          // hit target over the content, and — because `showPopover()` throws
+          // `InvalidStateError` on an already-showing popover — an uncaught error out of
+          // the next `mouseenter`, after which the tooltip never appeared again.
+          _this3.closePopover();
+          // Paired with the `visibility` that `show()` sets, so a popover left open by
+          // a failed close cannot intercept pointer events over the content.
+          _this3.tooltip.style.visibility = 'hidden';
+        } else {
+          var _this3$popperInstance2;
+          (_this3$popperInstance2 = _this3.popperInstance) === null || _this3$popperInstance2 === void 0 || _this3$popperInstance2.setOptions(function (options) {
+            return _objectSpread(_objectSpread({}, options), {}, {
+              modifiers: [startingOffset, {
+                name: 'eventListeners',
+                enabled: false
+              }]
+            });
           });
-        });
-        _this3.tooltip.style.opacity = '0';
-        _this3.isDisplayed = false;
-        // If transition animations are disabled immediately hide the popover
-        if (_this3.tooltip.style.transition === 'none') {
-          _this3.tooltip.hidePopover();
+          _this3.tooltip.style.opacity = '0';
+          _this3.isDisplayed = false;
+          // If transition animations are disabled immediately hide the popover
+          if (_this3.tooltip.style.transition === 'none') {
+            _this3.tooltip.hidePopover();
+          }
         }
       }, immediate ? 0 : this.timeout);
     }
@@ -257,18 +320,26 @@ export var VanillaTooltip = /*#__PURE__*/function () {
     key: "show",
     value: function show() {
       var _this4 = this;
-      // Bound before the open delay so Escape can dismiss during it, and before the
-      // `isDisplayed` guard so re-hovering a tooltip whose hide is still pending (`hide()`
-      // unbinds synchronously, but the tooltip stays on screen until its timeout) restores
-      // Escape rather than leaving the visible tooltip undismissable.
+      // Ahead of the `isDisplayed` guard, unlike the equivalent below: `hide()` takes effect
+      // synchronously but leaves the tooltip on screen until its timeout, so a pointer
+      // returning in that window finds `isDisplayed` still true and returns early. Cancelling
+      // first keeps the tooltip up and stops it being left undismissable with a hide pending.
       if (isExperimentEnabled('platform_editor_use_vanilla_components')) {
         this.bindEscapeListener();
+        clearTimeout(this.currentTimeoutId);
+        this.shouldHidePopover = false;
       }
       if (this.isDisplayed) {
         return;
       }
-      clearTimeout(this.currentTimeoutId);
-      this.shouldHidePopover = false;
+
+      // Control only: the experiment cancels above, ahead of the guard. Kept gated rather than
+      // left unconditional — despite being a no-op once the block above has run — so that
+      // experiment cleanup can delete it outright instead of having to reason about it.
+      if (!isExperimentEnabled('platform_editor_use_vanilla_components')) {
+        clearTimeout(this.currentTimeoutId);
+        this.shouldHidePopover = false;
+      }
 
       // Make the tooltip visible - but hide until
       this.tooltip.style.visibility = 'hidden';

package/dist/types/vanilla-tooltip/index.d.ts

@@ -76,6 +76,15 @@ export declare class VanillaTooltip {
      * the `onShow` semantics of "@atlaskit/tooltip".
      */
     onShow?: (() => void) | undefined);
+    /**
+     * Closes the popover, tolerating it already being closed.
+     *
+     * `hidePopover()` throws `InvalidStateError` when the popover is not showing, and that is a
+     * routine state rather than an exceptional one: showing any other `hint` or `auto` popover
+     * light-dismisses this one without notifying us, so the caller cannot know whether it is
+     * still open.
+     */
+    private closePopover;
     private createPopperInstance;
     destroy(): void;
     /**

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@atlaskit/editor-common",
-	"version": "120.11.0",
+	"version": "120.11.1",
 	"description": "A package that contains common classes and components for editor and renderer",
 	"publishConfig": {
 		"registry": "https://registry.npmjs.org/"
@@ -54,7 +54,7 @@
 		"@atlaskit/feature-gate-js-client": "^6.0.0",
 		"@atlaskit/icon": "^37.6.0",
 		"@atlaskit/link": "^5.1.0",
-		"@atlaskit/link-datasource": "^6.9.0",
+		"@atlaskit/link-datasource": "^6.10.0",
 		"@atlaskit/link-picker": "^6.6.0",
 		"@atlaskit/media-card": "^81.9.0",
 		"@atlaskit/media-client": "^37.6.0",