@atlaskit/tooltip 17.5.13 → 17.5.16

package/CHANGELOG.md

@@ -1,5 +1,24 @@
 # @atlaskit/tooltip
 
+## 17.5.16
+
+### Patch Changes
+
+- [`8d4228767b0`](https://bitbucket.org/atlassian/atlassian-frontend/commits/8d4228767b0) - Upgrade Typescript from `4.2.4` to `4.3.5`.
+
+## 17.5.15
+
+### Patch Changes
+
+- [`338ba1a1500`](https://bitbucket.org/atlassian/atlassian-frontend/commits/338ba1a1500) - [ux] Improves accuracy of Tooltip placement relative to the mouse when using `position="mouse"`
+
+## 17.5.14
+
+### Patch Changes
+
+- [`13a202fde6b`](https://bitbucket.org/atlassian/atlassian-frontend/commits/13a202fde6b) - Fixes a bug where undefined test IDs were being added to Tooltip wrappers when no test ID was set
+- Updated dependencies
+
 ## 17.5.13
 
 ### Patch Changes

package/dist/cjs/Tooltip.js

@@ -23,6 +23,8 @@ var _bindEventListener = require("bind-event-listener");
 
 var _analyticsNext = require("@atlaskit/analytics-next");
 
+var _noop = _interopRequireDefault(require("@atlaskit/ds-lib/noop"));
+
 var _useCloseOnEscapePress = _interopRequireDefault(require("@atlaskit/ds-lib/use-close-on-escape-press"));
 
 var _motion = require("@atlaskit/motion");
@@ -56,11 +58,8 @@ var tooltipZIndex = _constants.layers.tooltip();
 var analyticsAttributes = {
   componentName: 'tooltip',
   packageName: "@atlaskit/tooltip",
-  packageVersion: "17.5.13"
-};
-
-function noop() {} // Inverts motion direction
-
+  packageVersion: "17.5.16"
+}; // Inverts motion direction
 
 var invertedDirection = {
   top: 'bottom',
@@ -96,9 +95,9 @@ function Tooltip(_ref) {
       _ref$delay = _ref.delay,
       delay = _ref$delay === void 0 ? 300 : _ref$delay,
       _ref$onShow = _ref.onShow,
-      onShow = _ref$onShow === void 0 ? noop : _ref$onShow,
+      onShow = _ref$onShow === void 0 ? _noop.default : _ref$onShow,
       _ref$onHide = _ref.onHide,
-      onHide = _ref$onHide === void 0 ? noop : _ref$onHide,
+      onHide = _ref$onHide === void 0 ? _noop.default : _ref$onHide,
       _ref$hideTooltipOnCli = _ref.hideTooltipOnClick,
       hideTooltipOnClick = _ref$hideTooltipOnCli === void 0 ? false : _ref$hideTooltipOnCli,
       _ref$hideTooltipOnMou = _ref.hideTooltipOnMouseDown,
@@ -252,7 +251,7 @@ function Tooltip(_ref) {
   });
   (0, _react.useEffect)(function () {
     if (state === 'hide') {
-      return noop;
+      return _noop.default;
     }
 
     if (state === 'before-fade-out') {
@@ -343,6 +342,16 @@ function Tooltip(_ref) {
       });
     }
   }, []);
+  var onMouseMove = position === 'mouse' ? function (event) {
+    var _apiRef$current2;
+
+    if ((_apiRef$current2 = apiRef.current) !== null && _apiRef$current2 !== void 0 && _apiRef$current2.isActive()) {
+      apiRef.current.mousePosition = (0, _utilities.getMousePosition)({
+        left: event.clientX,
+        top: event.clientY
+      });
+    }
+  } : undefined;
   var onMouseOverTooltip = (0, _react.useCallback)(function () {
     if (apiRef.current && apiRef.current.isActive()) {
       apiRef.current.keep();
@@ -374,12 +383,12 @@ function Tooltip(_ref) {
   var shouldRenderTooltipChildren = state !== 'hide' && state !== 'fade-out';
 
   var getReferenceElement = function getReferenceElement() {
-    // Use the initial mouse position if appropriate, or the target element
-    var api = apiRef.current;
-    var initialMouse = api ? api.getInitialMouse() : null;
+    var _apiRef$current3;
+
+    if (position === 'mouse' && (_apiRef$current3 = apiRef.current) !== null && _apiRef$current3 !== void 0 && _apiRef$current3.mousePosition) {
+      var _apiRef$current4;
 
-    if (position === 'mouse' && initialMouse) {
-      return initialMouse;
+      return (_apiRef$current4 = apiRef.current) === null || _apiRef$current4 === void 0 ? void 0 : _apiRef$current4.mousePosition;
     }
 
     return targetRef.current || undefined;
@@ -389,6 +398,7 @@ function Tooltip(_ref) {
   var tooltipTriggerProps = {
     onMouseOver: onMouseOver,
     onMouseOut: onMouseOut,
+    onMouseMove: onMouseMove,
     onClick: onClick,
     onMouseDown: onMouseDown,
     onFocus: onFocus,

package/dist/cjs/TooltipPrimitive.js

@@ -39,7 +39,7 @@ var TooltipPrimitive = /*#__PURE__*/(0, _react.forwardRef)(function TooltipPrimi
   return (0, _core.jsx)("div", {
     ref: ref,
     style: styleWithZIndex,
-    "data-testid": "".concat(testId, "--wrapper")
+    "data-testid": testId ? "".concat(testId, "--wrapper") : undefined
   }, (0, _core.jsx)("div", {
     role: "tooltip",
     className: className,

package/dist/cjs/internal/tooltip-manager.js

@@ -157,7 +157,7 @@ function show(entry) {
     isActive: isActive,
     requestHide: requestHide,
     finishHideAnimation: finishHideAnimation,
-    getInitialMouse: getInitialMouse
+    mousePosition: getInitialMouse()
   };
   return result;
 }

package/dist/cjs/version.json

@@ -1,5 +1,5 @@
 {
 	"name": "@atlaskit/tooltip",
-	"version": "17.5.13",
+	"version": "17.5.16",
 	"sideEffects": false
 }

package/dist/es2019/Tooltip.js

@@ -5,6 +5,7 @@ import React, { useCallback, useEffect, useRef, useState } from 'react';
 import { jsx } from '@emotion/core';
 import { bind } from 'bind-event-listener';
 import { usePlatformLeafSyntheticEventHandler } from '@atlaskit/analytics-next';
+import noop from '@atlaskit/ds-lib/noop';
 import useCloseOnEscapePress from '@atlaskit/ds-lib/use-close-on-escape-press';
 import { ExitingPersistence, FadeIn } from '@atlaskit/motion';
 import { mediumDurationMs } from '@atlaskit/motion/durations';
@@ -19,11 +20,8 @@ const tooltipZIndex = layers.tooltip();
 const analyticsAttributes = {
   componentName: 'tooltip',
   packageName: "@atlaskit/tooltip",
-  packageVersion: "17.5.13"
-};
-
-function noop() {} // Inverts motion direction
-
+  packageVersion: "17.5.16"
+}; // Inverts motion direction
 
 const invertedDirection = {
   top: 'bottom',
@@ -291,6 +289,16 @@ function Tooltip({
       });
     }
   }, []);
+  const onMouseMove = position === 'mouse' ? event => {
+    var _apiRef$current2;
+
+    if ((_apiRef$current2 = apiRef.current) !== null && _apiRef$current2 !== void 0 && _apiRef$current2.isActive()) {
+      apiRef.current.mousePosition = getMousePosition({
+        left: event.clientX,
+        top: event.clientY
+      });
+    }
+  } : undefined;
   const onMouseOverTooltip = useCallback(() => {
     if (apiRef.current && apiRef.current.isActive()) {
       apiRef.current.keep();
@@ -322,12 +330,12 @@ function Tooltip({
   const shouldRenderTooltipChildren = state !== 'hide' && state !== 'fade-out';
 
   const getReferenceElement = () => {
-    // Use the initial mouse position if appropriate, or the target element
-    const api = apiRef.current;
-    const initialMouse = api ? api.getInitialMouse() : null;
+    var _apiRef$current3;
+
+    if (position === 'mouse' && (_apiRef$current3 = apiRef.current) !== null && _apiRef$current3 !== void 0 && _apiRef$current3.mousePosition) {
+      var _apiRef$current4;
 
-    if (position === 'mouse' && initialMouse) {
-      return initialMouse;
+      return (_apiRef$current4 = apiRef.current) === null || _apiRef$current4 === void 0 ? void 0 : _apiRef$current4.mousePosition;
     }
 
     return targetRef.current || undefined;
@@ -337,6 +345,7 @@ function Tooltip({
   const tooltipTriggerProps = {
     onMouseOver,
     onMouseOut,
+    onMouseMove,
     onClick,
     onMouseDown,
     onFocus,

package/dist/es2019/TooltipPrimitive.js

@@ -23,7 +23,7 @@ const TooltipPrimitive = /*#__PURE__*/forwardRef(function TooltipPrimitive({
   return jsx("div", {
     ref: ref,
     style: styleWithZIndex,
-    "data-testid": `${testId}--wrapper`
+    "data-testid": testId ? `${testId}--wrapper` : undefined
   }, jsx("div", {
     role: "tooltip",
     className: className,

package/dist/es2019/internal/tooltip-manager.js

@@ -149,7 +149,7 @@ export function show(entry) {
     isActive,
     requestHide,
     finishHideAnimation,
-    getInitialMouse
+    mousePosition: getInitialMouse()
   };
   return result;
 }

package/dist/es2019/version.json

@@ -1,5 +1,5 @@
 {
 	"name": "@atlaskit/tooltip",
-	"version": "17.5.13",
+	"version": "17.5.16",
 	"sideEffects": false
 }

package/dist/esm/Tooltip.js

@@ -11,6 +11,7 @@ import React, { useCallback, useEffect, useRef, useState } from 'react';
 import { jsx } from '@emotion/core';
 import { bind } from 'bind-event-listener';
 import { usePlatformLeafSyntheticEventHandler } from '@atlaskit/analytics-next';
+import noop from '@atlaskit/ds-lib/noop';
 import useCloseOnEscapePress from '@atlaskit/ds-lib/use-close-on-escape-press';
 import { ExitingPersistence, FadeIn } from '@atlaskit/motion';
 import { mediumDurationMs } from '@atlaskit/motion/durations';
@@ -25,11 +26,8 @@ var tooltipZIndex = layers.tooltip();
 var analyticsAttributes = {
   componentName: 'tooltip',
   packageName: "@atlaskit/tooltip",
-  packageVersion: "17.5.13"
-};
-
-function noop() {} // Inverts motion direction
-
+  packageVersion: "17.5.16"
+}; // Inverts motion direction
 
 var invertedDirection = {
   top: 'bottom',
@@ -312,6 +310,16 @@ function Tooltip(_ref) {
       });
     }
   }, []);
+  var onMouseMove = position === 'mouse' ? function (event) {
+    var _apiRef$current2;
+
+    if ((_apiRef$current2 = apiRef.current) !== null && _apiRef$current2 !== void 0 && _apiRef$current2.isActive()) {
+      apiRef.current.mousePosition = getMousePosition({
+        left: event.clientX,
+        top: event.clientY
+      });
+    }
+  } : undefined;
   var onMouseOverTooltip = useCallback(function () {
     if (apiRef.current && apiRef.current.isActive()) {
       apiRef.current.keep();
@@ -343,12 +351,12 @@ function Tooltip(_ref) {
   var shouldRenderTooltipChildren = state !== 'hide' && state !== 'fade-out';
 
   var getReferenceElement = function getReferenceElement() {
-    // Use the initial mouse position if appropriate, or the target element
-    var api = apiRef.current;
-    var initialMouse = api ? api.getInitialMouse() : null;
+    var _apiRef$current3;
+
+    if (position === 'mouse' && (_apiRef$current3 = apiRef.current) !== null && _apiRef$current3 !== void 0 && _apiRef$current3.mousePosition) {
+      var _apiRef$current4;
 
-    if (position === 'mouse' && initialMouse) {
-      return initialMouse;
+      return (_apiRef$current4 = apiRef.current) === null || _apiRef$current4 === void 0 ? void 0 : _apiRef$current4.mousePosition;
     }
 
     return targetRef.current || undefined;
@@ -358,6 +366,7 @@ function Tooltip(_ref) {
   var tooltipTriggerProps = {
     onMouseOver: onMouseOver,
     onMouseOut: onMouseOut,
+    onMouseMove: onMouseMove,
     onClick: onClick,
     onMouseDown: onMouseDown,
     onFocus: onFocus,

package/dist/esm/TooltipPrimitive.js

@@ -28,7 +28,7 @@ var TooltipPrimitive = /*#__PURE__*/forwardRef(function TooltipPrimitive(_ref, r
   return jsx("div", {
     ref: ref,
     style: styleWithZIndex,
-    "data-testid": "".concat(testId, "--wrapper")
+    "data-testid": testId ? "".concat(testId, "--wrapper") : undefined
   }, jsx("div", {
     role: "tooltip",
     className: className,

package/dist/esm/internal/tooltip-manager.js

@@ -149,7 +149,7 @@ export function show(entry) {
     isActive: isActive,
     requestHide: requestHide,
     finishHideAnimation: finishHideAnimation,
-    getInitialMouse: getInitialMouse
+    mousePosition: getInitialMouse()
   };
   return result;
 }

package/dist/esm/version.json

@@ -1,5 +1,5 @@
 {
 	"name": "@atlaskit/tooltip",
-	"version": "17.5.13",
+	"version": "17.5.16",
 	"sideEffects": false
 }

package/dist/types/internal/tooltip-manager.d.ts

@@ -18,7 +18,7 @@ export declare type Entry = {
 };
 export declare type API = {
     isActive: () => boolean;
-    getInitialMouse: () => FakeMouseElement | null;
+    mousePosition: FakeMouseElement | null;
     requestHide: (value: {
         isImmediate: boolean;
     }) => void;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@atlaskit/tooltip",
-  "version": "17.5.13",
+  "version": "17.5.16",
   "description": "A tooltip is a floating, non-actionable label used to explain a user interface element or feature.",
   "publishConfig": {
     "registry": "https://registry.npmjs.org/"
@@ -32,7 +32,7 @@
   },
   "dependencies": {
     "@atlaskit/analytics-next": "^8.2.0",
-    "@atlaskit/ds-lib": "^2.0.0",
+    "@atlaskit/ds-lib": "^2.1.0",
     "@atlaskit/motion": "^1.2.0",
     "@atlaskit/popper": "^5.0.0",
     "@atlaskit/portal": "^4.1.0",
@@ -63,7 +63,7 @@
     "react-dom": "^16.8.0",
     "react-redux": "^5.1.2",
     "storybook-addon-performance": "^0.16.0",
-    "typescript": "4.2.4",
+    "typescript": "4.3.5",
     "wait-for-expect": "^1.2.0"
   },
   "keywords": [