@atlaskit/reactions 21.3.1 → 21.3.4

package/CHANGELOG.md

@@ -1,5 +1,25 @@
 # @atlaskit/reactions
 
+## 21.3.4
+
+### Patch Changes
+
+- [`c3f9e9bce1c`](https://bitbucket.org/atlassian/atlassian-frontend/commits/c3f9e9bce1c) - Add custom information to failure and abort events for UFO experiences
+- Updated dependencies
+
+## 21.3.3
+
+### Patch Changes
+
+- [`7f1dd280229`](https://bitbucket.org/atlassian/atlassian-frontend/commits/7f1dd280229) - Fix styling issue in the reaction emoji while hovering on it for the first time
+
+## 21.3.2
+
+### Patch Changes
+
+- [`1940b04de7d`](https://bitbucket.org/atlassian/atlassian-frontend/commits/1940b04de7d) - Adding new UFO experience for contents list of reacted users should be able to be fetched reliably
+- Updated dependencies
+
 ## 21.3.1
 
 ### Patch Changes

package/dist/cjs/analytics/ufo.js

@@ -3,7 +3,7 @@
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
-exports.ReactionsRendered = exports.ReactionsRemove = exports.ReactionsAdd = exports.PickerRender = exports.ExperienceName = exports.ComponentName = void 0;
+exports.ReactionsRendered = exports.ReactionsRemove = exports.ReactionsAdd = exports.ReactionDetailsFetch = exports.PickerRender = exports.ExperienceName = exports.ComponentName = void 0;
 
 var _ufo = require("@atlaskit/ufo");
 
@@ -37,6 +37,7 @@ exports.ExperienceName = ExperienceName;
   ExperienceName["PICKER_OPENED"] = "reactions-picker-opened";
   ExperienceName["REACTION_ADDED"] = "reaction-added";
   ExperienceName["REACTION_REMOVED"] = "reaction-removed";
+  ExperienceName["REACTION_DETAILS_FETCHED"] = "reaction-details-fetched";
 })(ExperienceName || (exports.ExperienceName = ExperienceName = {}));
 
 var ComponentName;
@@ -49,6 +50,7 @@ exports.ComponentName = ComponentName;
 (function (ComponentName) {
   ComponentName["PICKER_RENDERED"] = "reactions-picker";
   ComponentName["REACTIONS"] = "reactions-list";
+  ComponentName["REACTION_ITEM"] = "reaction-item";
 })(ComponentName || (exports.ComponentName = ComponentName = {}));
 
 var PickerRender = new _ufo.UFOExperience(ExperienceName.PICKER_OPENED, createExperienceConfig(ComponentName.PICKER_RENDERED, _ufo.ExperienceTypes.Experience, _ufo.ExperiencePerformanceTypes.InlineResult));
@@ -65,9 +67,15 @@ var ReactionsRendered = new _ufo.ConcurrentExperience(ExperienceName.REACTIONS_R
 exports.ReactionsRendered = ReactionsRendered;
 var ReactionsAdd = new _ufo.ConcurrentExperience(ExperienceName.REACTION_ADDED, createExperienceConfig(ComponentName.REACTIONS, _ufo.ExperienceTypes.Experience, _ufo.ExperiencePerformanceTypes.InlineResult));
 /**
- * Experience when a reaction emoji gets removed/decrement
+ * Experience when a reaction details gets fetched
  */
 
 exports.ReactionsAdd = ReactionsAdd;
+var ReactionDetailsFetch = new _ufo.ConcurrentExperience(ExperienceName.REACTION_DETAILS_FETCHED, createExperienceConfig(ComponentName.REACTION_ITEM, _ufo.ExperienceTypes.Experience, _ufo.ExperiencePerformanceTypes.InlineResult));
+/**
+ * Experience when a reaction emoji gets removed/decrement
+ */
+
+exports.ReactionDetailsFetch = ReactionDetailsFetch;
 var ReactionsRemove = new _ufo.ConcurrentExperience(ExperienceName.REACTION_REMOVED, createExperienceConfig(ComponentName.REACTIONS, _ufo.ExperienceTypes.Experience, _ufo.ExperiencePerformanceTypes.InlineResult));
 exports.ReactionsRemove = ReactionsRemove;

package/dist/cjs/components/ReactionPicker.js

@@ -189,7 +189,12 @@ var ReactionPicker = /*#__PURE__*/function (_PureComponent) {
     value: function componentWillUnmount() {
       document.removeEventListener('click', this.handleClickOutside);
 
-      _analytics.UFO.PickerRender.abort();
+      _analytics.UFO.PickerRender.abort({
+        metadata: {
+          source: 'Reaction-Picker',
+          reason: 'unmount'
+        }
+      });
     }
   }, {
     key: "close",
@@ -199,7 +204,13 @@ var ReactionPicker = /*#__PURE__*/function (_PureComponent) {
         showFullPicker: false
       }, function () {
         // ufo abort reaction experience
-        _analytics.UFO.PickerRender.abort();
+        _analytics.UFO.PickerRender.abort({
+          metadata: {
+            emojiId: _emojiId,
+            source: 'Reaction-Picker',
+            reason: 'close dialog'
+          }
+        });
       });
     }
   }, {

package/dist/cjs/components/ReactionTooltip.js

@@ -2,8 +2,6 @@
 
 var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefault");
 
-var _typeof = require("@babel/runtime/helpers/typeof");
-
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
@@ -11,12 +9,12 @@ exports.ReactionTooltip = void 0;
 
 var _extends2 = _interopRequireDefault(require("@babel/runtime/helpers/extends"));
 
+var _react = _interopRequireDefault(require("react"));
+
 var _core = require("@emotion/core");
 
 var _tooltip = _interopRequireDefault(require("@atlaskit/tooltip"));
 
-var _react = _interopRequireWildcard(require("react"));
-
 var _reactIntlNext = require("react-intl-next");
 
 var _tokens = require("@atlaskit/tokens");
@@ -25,10 +23,6 @@ var _colors = require("@atlaskit/theme/colors");
 
 var _i18n = require("./i18n");
 
-function _getRequireWildcardCache(nodeInterop) { if (typeof WeakMap !== "function") return null; var cacheBabelInterop = new WeakMap(); var cacheNodeInterop = new WeakMap(); return (_getRequireWildcardCache = function _getRequireWildcardCache(nodeInterop) { return nodeInterop ? cacheNodeInterop : cacheBabelInterop; })(nodeInterop); }
-
-function _interopRequireWildcard(obj, nodeInterop) { if (!nodeInterop && obj && obj.__esModule) { return obj; } if (obj === null || _typeof(obj) !== "object" && typeof obj !== "function") { return { default: obj }; } var cache = _getRequireWildcardCache(nodeInterop); if (cache && cache.has(obj)) { return cache.get(obj); } var newObj = {}; var hasPropertyDescriptor = Object.defineProperty && Object.getOwnPropertyDescriptor; for (var key in obj) { if (key !== "default" && Object.prototype.hasOwnProperty.call(obj, key)) { var desc = hasPropertyDescriptor ? Object.getOwnPropertyDescriptor(obj, key) : null; if (desc && (desc.get || desc.set)) { Object.defineProperty(newObj, key, desc); } else { newObj[key] = obj[key]; } } } newObj.default = obj; if (cache) { cache.set(obj, newObj); } return newObj; }
-
 /** @jsx jsx */
 var verticalMargin = 5;
 var tooltipStyle = (0, _core.css)({
@@ -64,12 +58,7 @@ var ReactionTooltip = function ReactionTooltip(_ref) {
   var emojiName = _ref.emojiName,
       children = _ref.children,
       users = _ref.reaction.users;
-
-  if (!users || users.length === 0) {
-    return (0, _core.jsx)(_react.Fragment, null, _react.default.Children.only(children));
-  }
-
-  var content = (0, _core.jsx)("div", {
+  var content = !users || users.length === 0 ? null : (0, _core.jsx)("div", {
     css: tooltipStyle
   }, (0, _core.jsx)("ul", null, emojiName ? (0, _core.jsx)("li", {
     css: emojiNameStyle

package/dist/cjs/components/Reactions.js

@@ -50,7 +50,7 @@ var reactionStyle = (0, _core.css)({
   // top margin of 2px to allow spacing between rows when wrapped (paired with top margin in reactionsStyle)
   margin: '2px 4px 0 4px'
 });
-var reactionsStyle = (0, _core.css)({
+var wrapperStyle = (0, _core.css)({
   display: 'flex',
   flexWrap: 'wrap',
   position: 'relative',
@@ -58,7 +58,7 @@ var reactionsStyle = (0, _core.css)({
   borderRadius: '15px',
   // To allow to row spacing of 2px on wrap, and 0px on first row
   marginTop: '-2px',
-  '& > :first-of-type': {
+  '> :first-of-type > :first-child': {
     marginLeft: 0
   }
 });
@@ -177,7 +177,7 @@ var ReactionsWithoutAnalytics = /*#__PURE__*/function (_React$PureComponent) {
     key: "render",
     value: function render() {
       return (0, _core.jsx)("div", {
-        css: reactionsStyle
+        css: wrapperStyle
       }, this.props.reactions.map(this.renderReaction), this.renderPicker());
     }
   }]);

package/dist/cjs/components/UfoErrorBoundary.js

@@ -41,14 +41,24 @@ var UfoErrorBoundary = /*#__PURE__*/function (_React$Component) {
 
   (0, _createClass2.default)(UfoErrorBoundary, [{
     key: "componentDidCatch",
-    value: function componentDidCatch() {
+    value: function componentDidCatch(error, errorInfo) {
       var _iterator = _createForOfIteratorHelper(this.props.experiences),
           _step;
 
       try {
         for (_iterator.s(); !(_step = _iterator.n()).done;) {
-          var e = _step.value;
-          e.failure();
+          var exp = _step.value;
+          exp.failure({
+            metadata: {
+              source: 'UfoErrorBoundary',
+              reason: 'error',
+              error: {
+                name: error.name,
+                message: error.message,
+                infoStack: errorInfo.componentStack
+              }
+            }
+          });
         }
       } catch (err) {
         _iterator.e(err);

package/dist/cjs/containers/ConnectedReactionsView.js

@@ -34,7 +34,8 @@ var ReactionsView = function ReactionsView(props) {
   var experienceInstance = (0, _react.useRef)();
   var ari = props.ari,
       createAnalyticsEvent = props.createAnalyticsEvent,
-      store = props.store;
+      store = props.store,
+      containerAri = props.containerAri;
   (0, _react.useEffect)(function () {
     experienceInstance.current = _ReactionsStore.ufoExperiences.render.getInstance(ari);
   }, [ari]);
@@ -47,16 +48,23 @@ var ReactionsView = function ReactionsView(props) {
   }, [createAnalyticsEvent, store]); // abort when component gets unmounted
 
   (0, _react.useEffect)(function () {
-    return function () {
+    return function cleanup() {
       var _experienceInstance$c;
 
-      (_experienceInstance$c = experienceInstance.current) === null || _experienceInstance$c === void 0 ? void 0 : _experienceInstance$c.abort();
+      (_experienceInstance$c = experienceInstance.current) === null || _experienceInstance$c === void 0 ? void 0 : _experienceInstance$c.abort({
+        metadata: {
+          source: 'Connected-Reactions-View',
+          data: {
+            ari: ari,
+            containerAri: containerAri
+          },
+          reason: 'unmount'
+        }
+      });
     };
-  }, [experienceInstance]);
+  }, [experienceInstance, containerAri, ari]);
 
   var renderChildren = function renderChildren(innerProps) {
-    var containerAri = props.containerAri,
-        ari = props.ari;
     return /*#__PURE__*/_react.default.createElement(_analyticsNamespacedContext.FabricElementsAnalyticsContext, {
       data: {
         containerAri: containerAri,

package/dist/cjs/store/ReactionsStore.js

@@ -35,9 +35,25 @@ function _objectSpread(target) { for (var i = 1; i < arguments.length; i++) { va
  * Set of all available UFO experiences relating to reaction element
  */
 var ufoExperiences = {
+  /**
+   * Experience when a reaction emoji gets added
+   */
   add: Analytics.UFO.ReactionsAdd,
+
+  /**
+   * Experience when a reaction emoji gets removed/decrement
+   */
   remove: Analytics.UFO.ReactionsRemove,
-  render: Analytics.UFO.ReactionsRendered
+
+  /**
+   * Experience when the list of reactions gets rendered
+   */
+  render: Analytics.UFO.ReactionsRendered,
+
+  /**
+   * Experience when a reaction details gets fetched
+   */
+  fetchDetails: Analytics.UFO.ReactionDetailsFetch
 };
 exports.ufoExperiences = ufoExperiences;
 
@@ -150,7 +166,15 @@ var MemoryReactionsStore = /*#__PURE__*/function () {
 
           exp.failure({
             metadata: {
-              error: error
+              error: error,
+              source: 'Reaction-Store',
+              data: {
+                containerAri: containerAri,
+                ari: ari,
+                emojiId: emojiId,
+                metadata: _this.metadata
+              },
+              reason: 'addReaction fetch failed'
             }
           });
         }
@@ -175,7 +199,15 @@ var MemoryReactionsStore = /*#__PURE__*/function () {
         // ufo add reaction failure
         exp.failure({
           metadata: {
-            error: error
+            error: error,
+            source: 'Reaction-Store',
+            data: {
+              containerAri: containerAri,
+              ari: ari,
+              emojiId: emojiId,
+              metadata: _this.metadata
+            },
+            reason: 'deleteReaction fetch failed'
           }
         });
       });
@@ -188,11 +220,12 @@ var MemoryReactionsStore = /*#__PURE__*/function () {
        * TODO:
        * All reactions are usually fetched in a single call to reactions-service. Need to check why "getReactions" gets called randomly 1-2 times everytime on each fetch request despite using same containerAri.
        */
-      var exp = ufoExperiences.render.getInstance(containerAri); // ufo start reaction experience
+      var exp = ufoExperiences.render.getInstance(containerAri);
+      var arisArr = aris.reduce(utils.flattenAris); // ufo start reaction experience
 
       exp.start();
 
-      _this.client.getReactions(containerAri, aris.reduce(utils.flattenAris)).then(function (value) {
+      _this.client.getReactions(containerAri, arisArr).then(function (value) {
         Object.keys(value).map(function (ari) {
           var reactionsState = _this.getReactionsState(containerAri, ari);
 
@@ -212,7 +245,17 @@ var MemoryReactionsStore = /*#__PURE__*/function () {
             Analytics.createAndFireSafe(_this.createAnalyticsEvent, Analytics.createRestFailedEvent, 'getReactions', error.code);
           }
 
-          exp.failure();
+          exp.failure({
+            metadata: {
+              error: error,
+              source: 'Reaction-Store',
+              data: {
+                containerAri: containerAri,
+                aris: arisArr.join(',')
+              },
+              reason: 'getReactions fetch failed'
+            }
+          });
           return Promise.reject(error);
         }
       });
@@ -220,7 +263,29 @@ var MemoryReactionsStore = /*#__PURE__*/function () {
     (0, _defineProperty2.default)(this, "toggleReaction", this.withReaction(this.doRemoveReaction, this.doAddReaction));
     (0, _defineProperty2.default)(this, "addReaction", this.withReaction(this.flash, this.doAddReaction));
     (0, _defineProperty2.default)(this, "getDetailedReaction", function (containerAri, ari, emojiId) {
-      _this.client.getDetailedReaction(containerAri, ari, emojiId).then(_this.handleDetailedReactionResponse);
+      var exp = ufoExperiences.fetchDetails.getInstance("".concat(ari, "|").concat(emojiId));
+      exp.start();
+
+      _this.client.getDetailedReaction(containerAri, ari, emojiId).then(function (summary) {
+        // ufo get reaction details success
+        exp.success();
+
+        _this.handleDetailedReactionResponse(summary);
+      }).catch(function (error) {
+        // ufo get reaction details failure
+        exp.failure({
+          metadata: {
+            error: error,
+            source: 'Reaction-Store',
+            data: {
+              containerAri: containerAri,
+              ari: ari,
+              emojiId: emojiId
+            },
+            reason: 'getDetailedReaction fetch failed'
+          }
+        });
+      });
     });
     (0, _defineProperty2.default)(this, "getState", function () {
       return _this.state;

package/dist/cjs/version.json

@@ -1,4 +1,4 @@
 {
 	"name": "@atlaskit/reactions",
-	"version": "21.3.1"
+	"version": "21.3.4"
 }

package/dist/es2019/analytics/ufo.js

@@ -32,6 +32,7 @@ export let ExperienceName;
   ExperienceName["PICKER_OPENED"] = "reactions-picker-opened";
   ExperienceName["REACTION_ADDED"] = "reaction-added";
   ExperienceName["REACTION_REMOVED"] = "reaction-removed";
+  ExperienceName["REACTION_DETAILS_FETCHED"] = "reaction-details-fetched";
 })(ExperienceName || (ExperienceName = {}));
 
 export let ComponentName;
@@ -42,6 +43,7 @@ export let ComponentName;
 (function (ComponentName) {
   ComponentName["PICKER_RENDERED"] = "reactions-picker";
   ComponentName["REACTIONS"] = "reactions-list";
+  ComponentName["REACTION_ITEM"] = "reaction-item";
 })(ComponentName || (ComponentName = {}));
 
 export const PickerRender = new UFOExperience(ExperienceName.PICKER_OPENED, createExperienceConfig(ComponentName.PICKER_RENDERED, ExperienceTypes.Experience, ExperiencePerformanceTypes.InlineResult));
@@ -55,6 +57,11 @@ export const ReactionsRendered = new ConcurrentExperience(ExperienceName.REACTIO
  */
 
 export const ReactionsAdd = new ConcurrentExperience(ExperienceName.REACTION_ADDED, createExperienceConfig(ComponentName.REACTIONS, ExperienceTypes.Experience, ExperiencePerformanceTypes.InlineResult));
+/**
+ * Experience when a reaction details gets fetched
+ */
+
+export const ReactionDetailsFetch = new ConcurrentExperience(ExperienceName.REACTION_DETAILS_FETCHED, createExperienceConfig(ComponentName.REACTION_ITEM, ExperienceTypes.Experience, ExperiencePerformanceTypes.InlineResult));
 /**
  * Experience when a reaction emoji gets removed/decrement
  */

package/dist/es2019/components/ReactionPicker.js

@@ -143,7 +143,12 @@ export class ReactionPicker extends PureComponent {
 
   componentWillUnmount() {
     document.removeEventListener('click', this.handleClickOutside);
-    UFO.PickerRender.abort();
+    UFO.PickerRender.abort({
+      metadata: {
+        source: 'Reaction-Picker',
+        reason: 'unmount'
+      }
+    });
   }
 
   close(_emojiId) {
@@ -152,7 +157,13 @@ export class ReactionPicker extends PureComponent {
       showFullPicker: false
     }, () => {
       // ufo abort reaction experience
-      UFO.PickerRender.abort();
+      UFO.PickerRender.abort({
+        metadata: {
+          emojiId: _emojiId,
+          source: 'Reaction-Picker',
+          reason: 'close dialog'
+        }
+      });
     });
   }
 

package/dist/es2019/components/ReactionTooltip.js

@@ -1,9 +1,9 @@
 import _extends from "@babel/runtime/helpers/extends";
 
 /** @jsx jsx */
+import React from 'react';
 import { jsx, css } from '@emotion/core';
 import Tooltip from '@atlaskit/tooltip';
-import React, { Fragment } from 'react';
 import { FormattedMessage } from 'react-intl-next';
 import { token } from '@atlaskit/tokens';
 import { N90 } from '@atlaskit/theme/colors';
@@ -44,11 +44,7 @@ export const ReactionTooltip = ({
     users
   }
 }) => {
-  if (!users || users.length === 0) {
-    return jsx(Fragment, null, React.Children.only(children));
-  }
-
-  const content = jsx("div", {
+  const content = !users || users.length === 0 ? null : jsx("div", {
     css: tooltipStyle
   }, jsx("ul", null, emojiName ? jsx("li", {
     css: emojiNameStyle

package/dist/es2019/components/Reactions.js

@@ -16,7 +16,7 @@ const reactionStyle = css({
   // top margin of 2px to allow spacing between rows when wrapped (paired with top margin in reactionsStyle)
   margin: '2px 4px 0 4px'
 });
-const reactionsStyle = css({
+const wrapperStyle = css({
   display: 'flex',
   flexWrap: 'wrap',
   position: 'relative',
@@ -24,7 +24,7 @@ const reactionsStyle = css({
   borderRadius: '15px',
   // To allow to row spacing of 2px on wrap, and 0px on first row
   marginTop: '-2px',
-  '& > :first-of-type': {
+  '> :first-of-type > :first-child': {
     marginLeft: 0
   }
 });
@@ -134,7 +134,7 @@ export class ReactionsWithoutAnalytics extends React.PureComponent {
 
   render() {
     return jsx("div", {
-      css: reactionsStyle
+      css: wrapperStyle
     }, this.props.reactions.map(this.renderReaction), this.renderPicker());
   }
 

package/dist/es2019/components/UfoErrorBoundary.js

@@ -1,8 +1,18 @@
 import React from 'react';
 export class UfoErrorBoundary extends React.Component {
-  componentDidCatch() {
-    for (const e of this.props.experiences) {
-      e.failure();
+  componentDidCatch(error, errorInfo) {
+    for (const exp of this.props.experiences) {
+      exp.failure({
+        metadata: {
+          source: 'UfoErrorBoundary',
+          reason: 'error',
+          error: {
+            name: error.name,
+            message: error.message,
+            infoStack: errorInfo.componentStack
+          }
+        }
+      });
     }
   }
 

package/dist/es2019/containers/ConnectedReactionsView.js

@@ -13,7 +13,8 @@ const ReactionsView = props => {
   const {
     ari,
     createAnalyticsEvent,
-    store
+    store,
+    containerAri
   } = props;
   useEffect(() => {
     experienceInstance.current = ufoExperiences.render.getInstance(ari);
@@ -27,18 +28,23 @@ const ReactionsView = props => {
   }, [createAnalyticsEvent, store]); // abort when component gets unmounted
 
   useEffect(() => {
-    return () => {
+    return function cleanup() {
       var _experienceInstance$c;
 
-      (_experienceInstance$c = experienceInstance.current) === null || _experienceInstance$c === void 0 ? void 0 : _experienceInstance$c.abort();
+      (_experienceInstance$c = experienceInstance.current) === null || _experienceInstance$c === void 0 ? void 0 : _experienceInstance$c.abort({
+        metadata: {
+          source: 'Connected-Reactions-View',
+          data: {
+            ari,
+            containerAri
+          },
+          reason: 'unmount'
+        }
+      });
     };
-  }, [experienceInstance]);
+  }, [experienceInstance, containerAri, ari]);
 
   const renderChildren = innerProps => {
-    const {
-      containerAri,
-      ari
-    } = props;
     return /*#__PURE__*/React.createElement(FabricElementsAnalyticsContext, {
       data: {
         containerAri,

package/dist/es2019/store/ReactionsStore.js

@@ -9,9 +9,25 @@ import { isRealErrorFromService } from './utils';
  * Set of all available UFO experiences relating to reaction element
  */
 export const ufoExperiences = {
+  /**
+   * Experience when a reaction emoji gets added
+   */
   add: Analytics.UFO.ReactionsAdd,
+
+  /**
+   * Experience when a reaction emoji gets removed/decrement
+   */
   remove: Analytics.UFO.ReactionsRemove,
-  render: Analytics.UFO.ReactionsRendered
+
+  /**
+   * Experience when the list of reactions gets rendered
+   */
+  render: Analytics.UFO.ReactionsRendered,
+
+  /**
+   * Experience when a reaction details gets fetched
+   */
+  fetchDetails: Analytics.UFO.ReactionDetailsFetch
 };
 export class MemoryReactionsStore {
   constructor(client, state = {
@@ -115,7 +131,15 @@ export class MemoryReactionsStore {
 
           exp.failure({
             metadata: {
-              error
+              error,
+              source: 'Reaction-Store',
+              data: {
+                containerAri,
+                ari,
+                emojiId,
+                metadata: this.metadata
+              },
+              reason: 'addReaction fetch failed'
             }
           });
         }
@@ -141,7 +165,15 @@ export class MemoryReactionsStore {
         // ufo add reaction failure
         exp.failure({
           metadata: {
-            error
+            error,
+            source: 'Reaction-Store',
+            data: {
+              containerAri,
+              ari,
+              emojiId,
+              metadata: this.metadata
+            },
+            reason: 'deleteReaction fetch failed'
           }
         });
       });
@@ -156,10 +188,11 @@ export class MemoryReactionsStore {
        * TODO:
        * All reactions are usually fetched in a single call to reactions-service. Need to check why "getReactions" gets called randomly 1-2 times everytime on each fetch request despite using same containerAri.
        */
-      const exp = ufoExperiences.render.getInstance(containerAri); // ufo start reaction experience
+      const exp = ufoExperiences.render.getInstance(containerAri);
+      const arisArr = aris.reduce(utils.flattenAris); // ufo start reaction experience
 
       exp.start();
-      this.client.getReactions(containerAri, aris.reduce(utils.flattenAris)).then(value => {
+      this.client.getReactions(containerAri, arisArr).then(value => {
         Object.keys(value).map(ari => {
           const reactionsState = this.getReactionsState(containerAri, ari);
           const reactions = reactionsState && reactionsState.status === Types.ReactionStatus.ready ? reactionsState.reactions : undefined;
@@ -177,7 +210,17 @@ export class MemoryReactionsStore {
             Analytics.createAndFireSafe(this.createAnalyticsEvent, Analytics.createRestFailedEvent, 'getReactions', error.code);
           }
 
-          exp.failure();
+          exp.failure({
+            metadata: {
+              error,
+              source: 'Reaction-Store',
+              data: {
+                containerAri,
+                aris: arisArr.join(',')
+              },
+              reason: 'getReactions fetch failed'
+            }
+          });
           return Promise.reject(error);
         }
       });
@@ -188,7 +231,27 @@ export class MemoryReactionsStore {
     _defineProperty(this, "addReaction", this.withReaction(this.flash, this.doAddReaction));
 
     _defineProperty(this, "getDetailedReaction", (containerAri, ari, emojiId) => {
-      this.client.getDetailedReaction(containerAri, ari, emojiId).then(this.handleDetailedReactionResponse);
+      const exp = ufoExperiences.fetchDetails.getInstance(`${ari}|${emojiId}`);
+      exp.start();
+      this.client.getDetailedReaction(containerAri, ari, emojiId).then(summary => {
+        // ufo get reaction details success
+        exp.success();
+        this.handleDetailedReactionResponse(summary);
+      }).catch(error => {
+        // ufo get reaction details failure
+        exp.failure({
+          metadata: {
+            error,
+            source: 'Reaction-Store',
+            data: {
+              containerAri,
+              ari,
+              emojiId
+            },
+            reason: 'getDetailedReaction fetch failed'
+          }
+        });
+      });
     });
 
     _defineProperty(this, "getState", () => this.state);

package/dist/es2019/version.json

@@ -1,4 +1,4 @@
 {
 	"name": "@atlaskit/reactions",
-	"version": "21.3.1"
+	"version": "21.3.4"
 }

package/dist/esm/analytics/ufo.js

@@ -32,6 +32,7 @@ export var ExperienceName;
   ExperienceName["PICKER_OPENED"] = "reactions-picker-opened";
   ExperienceName["REACTION_ADDED"] = "reaction-added";
   ExperienceName["REACTION_REMOVED"] = "reaction-removed";
+  ExperienceName["REACTION_DETAILS_FETCHED"] = "reaction-details-fetched";
 })(ExperienceName || (ExperienceName = {}));
 
 export var ComponentName;
@@ -42,6 +43,7 @@ export var ComponentName;
 (function (ComponentName) {
   ComponentName["PICKER_RENDERED"] = "reactions-picker";
   ComponentName["REACTIONS"] = "reactions-list";
+  ComponentName["REACTION_ITEM"] = "reaction-item";
 })(ComponentName || (ComponentName = {}));
 
 export var PickerRender = new UFOExperience(ExperienceName.PICKER_OPENED, createExperienceConfig(ComponentName.PICKER_RENDERED, ExperienceTypes.Experience, ExperiencePerformanceTypes.InlineResult));
@@ -55,6 +57,11 @@ export var ReactionsRendered = new ConcurrentExperience(ExperienceName.REACTIONS
  */
 
 export var ReactionsAdd = new ConcurrentExperience(ExperienceName.REACTION_ADDED, createExperienceConfig(ComponentName.REACTIONS, ExperienceTypes.Experience, ExperiencePerformanceTypes.InlineResult));
+/**
+ * Experience when a reaction details gets fetched
+ */
+
+export var ReactionDetailsFetch = new ConcurrentExperience(ExperienceName.REACTION_DETAILS_FETCHED, createExperienceConfig(ComponentName.REACTION_ITEM, ExperienceTypes.Experience, ExperiencePerformanceTypes.InlineResult));
 /**
  * Experience when a reaction emoji gets removed/decrement
  */

package/dist/esm/components/ReactionPicker.js

@@ -166,7 +166,12 @@ export var ReactionPicker = /*#__PURE__*/function (_PureComponent) {
     key: "componentWillUnmount",
     value: function componentWillUnmount() {
       document.removeEventListener('click', this.handleClickOutside);
-      UFO.PickerRender.abort();
+      UFO.PickerRender.abort({
+        metadata: {
+          source: 'Reaction-Picker',
+          reason: 'unmount'
+        }
+      });
     }
   }, {
     key: "close",
@@ -176,7 +181,13 @@ export var ReactionPicker = /*#__PURE__*/function (_PureComponent) {
         showFullPicker: false
       }, function () {
         // ufo abort reaction experience
-        UFO.PickerRender.abort();
+        UFO.PickerRender.abort({
+          metadata: {
+            emojiId: _emojiId,
+            source: 'Reaction-Picker',
+            reason: 'close dialog'
+          }
+        });
       });
     }
   }, {

package/dist/esm/components/ReactionTooltip.js

@@ -1,9 +1,9 @@
 import _extends from "@babel/runtime/helpers/extends";
 
 /** @jsx jsx */
+import React from 'react';
 import { jsx, css } from '@emotion/core';
 import Tooltip from '@atlaskit/tooltip';
-import React, { Fragment } from 'react';
 import { FormattedMessage } from 'react-intl-next';
 import { token } from '@atlaskit/tokens';
 import { N90 } from '@atlaskit/theme/colors';
@@ -41,12 +41,7 @@ export var ReactionTooltip = function ReactionTooltip(_ref) {
   var emojiName = _ref.emojiName,
       children = _ref.children,
       users = _ref.reaction.users;
-
-  if (!users || users.length === 0) {
-    return jsx(Fragment, null, React.Children.only(children));
-  }
-
-  var content = jsx("div", {
+  var content = !users || users.length === 0 ? null : jsx("div", {
     css: tooltipStyle
   }, jsx("ul", null, emojiName ? jsx("li", {
     css: emojiNameStyle

package/dist/esm/components/Reactions.js

@@ -26,7 +26,7 @@ var reactionStyle = css({
   // top margin of 2px to allow spacing between rows when wrapped (paired with top margin in reactionsStyle)
   margin: '2px 4px 0 4px'
 });
-var reactionsStyle = css({
+var wrapperStyle = css({
   display: 'flex',
   flexWrap: 'wrap',
   position: 'relative',
@@ -34,7 +34,7 @@ var reactionsStyle = css({
   borderRadius: '15px',
   // To allow to row spacing of 2px on wrap, and 0px on first row
   marginTop: '-2px',
-  '& > :first-of-type': {
+  '> :first-of-type > :first-child': {
     marginLeft: 0
   }
 });
@@ -162,7 +162,7 @@ export var ReactionsWithoutAnalytics = /*#__PURE__*/function (_React$PureCompone
     key: "render",
     value: function render() {
       return jsx("div", {
-        css: reactionsStyle
+        css: wrapperStyle
       }, this.props.reactions.map(this.renderReaction), this.renderPicker());
     }
   }]);

package/dist/esm/components/UfoErrorBoundary.js

@@ -28,14 +28,24 @@ export var UfoErrorBoundary = /*#__PURE__*/function (_React$Component) {
 
   _createClass(UfoErrorBoundary, [{
     key: "componentDidCatch",
-    value: function componentDidCatch() {
+    value: function componentDidCatch(error, errorInfo) {
       var _iterator = _createForOfIteratorHelper(this.props.experiences),
           _step;
 
       try {
         for (_iterator.s(); !(_step = _iterator.n()).done;) {
-          var e = _step.value;
-          e.failure();
+          var exp = _step.value;
+          exp.failure({
+            metadata: {
+              source: 'UfoErrorBoundary',
+              reason: 'error',
+              error: {
+                name: error.name,
+                message: error.message,
+                infoStack: errorInfo.componentStack
+              }
+            }
+          });
         }
       } catch (err) {
         _iterator.e(err);

package/dist/esm/containers/ConnectedReactionsView.js

@@ -12,7 +12,8 @@ var ReactionsView = function ReactionsView(props) {
   var experienceInstance = useRef();
   var ari = props.ari,
       createAnalyticsEvent = props.createAnalyticsEvent,
-      store = props.store;
+      store = props.store,
+      containerAri = props.containerAri;
   useEffect(function () {
     experienceInstance.current = ufoExperiences.render.getInstance(ari);
   }, [ari]);
@@ -25,16 +26,23 @@ var ReactionsView = function ReactionsView(props) {
   }, [createAnalyticsEvent, store]); // abort when component gets unmounted
 
   useEffect(function () {
-    return function () {
+    return function cleanup() {
       var _experienceInstance$c;
 
-      (_experienceInstance$c = experienceInstance.current) === null || _experienceInstance$c === void 0 ? void 0 : _experienceInstance$c.abort();
+      (_experienceInstance$c = experienceInstance.current) === null || _experienceInstance$c === void 0 ? void 0 : _experienceInstance$c.abort({
+        metadata: {
+          source: 'Connected-Reactions-View',
+          data: {
+            ari: ari,
+            containerAri: containerAri
+          },
+          reason: 'unmount'
+        }
+      });
     };
-  }, [experienceInstance]);
+  }, [experienceInstance, containerAri, ari]);
 
   var renderChildren = function renderChildren(innerProps) {
-    var containerAri = props.containerAri,
-        ari = props.ari;
     return /*#__PURE__*/React.createElement(FabricElementsAnalyticsContext, {
       data: {
         containerAri: containerAri,

package/dist/esm/store/ReactionsStore.js

@@ -16,9 +16,25 @@ import { isRealErrorFromService } from './utils';
  * Set of all available UFO experiences relating to reaction element
  */
 export var ufoExperiences = {
+  /**
+   * Experience when a reaction emoji gets added
+   */
   add: Analytics.UFO.ReactionsAdd,
+
+  /**
+   * Experience when a reaction emoji gets removed/decrement
+   */
   remove: Analytics.UFO.ReactionsRemove,
-  render: Analytics.UFO.ReactionsRendered
+
+  /**
+   * Experience when the list of reactions gets rendered
+   */
+  render: Analytics.UFO.ReactionsRendered,
+
+  /**
+   * Experience when a reaction details gets fetched
+   */
+  fetchDetails: Analytics.UFO.ReactionDetailsFetch
 };
 export var MemoryReactionsStore = /*#__PURE__*/function () {
   function MemoryReactionsStore(client) {
@@ -138,7 +154,15 @@ export var MemoryReactionsStore = /*#__PURE__*/function () {
 
           exp.failure({
             metadata: {
-              error: error
+              error: error,
+              source: 'Reaction-Store',
+              data: {
+                containerAri: containerAri,
+                ari: ari,
+                emojiId: emojiId,
+                metadata: _this.metadata
+              },
+              reason: 'addReaction fetch failed'
             }
           });
         }
@@ -164,7 +188,15 @@ export var MemoryReactionsStore = /*#__PURE__*/function () {
         // ufo add reaction failure
         exp.failure({
           metadata: {
-            error: error
+            error: error,
+            source: 'Reaction-Store',
+            data: {
+              containerAri: containerAri,
+              ari: ari,
+              emojiId: emojiId,
+              metadata: _this.metadata
+            },
+            reason: 'deleteReaction fetch failed'
           }
         });
       });
@@ -179,11 +211,12 @@ export var MemoryReactionsStore = /*#__PURE__*/function () {
        * TODO:
        * All reactions are usually fetched in a single call to reactions-service. Need to check why "getReactions" gets called randomly 1-2 times everytime on each fetch request despite using same containerAri.
        */
-      var exp = ufoExperiences.render.getInstance(containerAri); // ufo start reaction experience
+      var exp = ufoExperiences.render.getInstance(containerAri);
+      var arisArr = aris.reduce(utils.flattenAris); // ufo start reaction experience
 
       exp.start();
 
-      _this.client.getReactions(containerAri, aris.reduce(utils.flattenAris)).then(function (value) {
+      _this.client.getReactions(containerAri, arisArr).then(function (value) {
         Object.keys(value).map(function (ari) {
           var reactionsState = _this.getReactionsState(containerAri, ari);
 
@@ -203,7 +236,17 @@ export var MemoryReactionsStore = /*#__PURE__*/function () {
             Analytics.createAndFireSafe(_this.createAnalyticsEvent, Analytics.createRestFailedEvent, 'getReactions', error.code);
           }
 
-          exp.failure();
+          exp.failure({
+            metadata: {
+              error: error,
+              source: 'Reaction-Store',
+              data: {
+                containerAri: containerAri,
+                aris: arisArr.join(',')
+              },
+              reason: 'getReactions fetch failed'
+            }
+          });
           return Promise.reject(error);
         }
       });
@@ -214,7 +257,29 @@ export var MemoryReactionsStore = /*#__PURE__*/function () {
     _defineProperty(this, "addReaction", this.withReaction(this.flash, this.doAddReaction));
 
     _defineProperty(this, "getDetailedReaction", function (containerAri, ari, emojiId) {
-      _this.client.getDetailedReaction(containerAri, ari, emojiId).then(_this.handleDetailedReactionResponse);
+      var exp = ufoExperiences.fetchDetails.getInstance("".concat(ari, "|").concat(emojiId));
+      exp.start();
+
+      _this.client.getDetailedReaction(containerAri, ari, emojiId).then(function (summary) {
+        // ufo get reaction details success
+        exp.success();
+
+        _this.handleDetailedReactionResponse(summary);
+      }).catch(function (error) {
+        // ufo get reaction details failure
+        exp.failure({
+          metadata: {
+            error: error,
+            source: 'Reaction-Store',
+            data: {
+              containerAri: containerAri,
+              ari: ari,
+              emojiId: emojiId
+            },
+            reason: 'getDetailedReaction fetch failed'
+          }
+        });
+      });
     });
 
     _defineProperty(this, "getState", function () {

package/dist/esm/version.json

@@ -1,4 +1,4 @@
 {
 	"name": "@atlaskit/reactions",
-	"version": "21.3.1"
+	"version": "21.3.4"
 }

package/dist/types/analytics/ufo.d.ts

@@ -6,14 +6,16 @@ export declare enum ExperienceName {
     REACTIONS_RENDERED = "reactions-rendered",
     PICKER_OPENED = "reactions-picker-opened",
     REACTION_ADDED = "reaction-added",
-    REACTION_REMOVED = "reaction-removed"
+    REACTION_REMOVED = "reaction-removed",
+    REACTION_DETAILS_FETCHED = "reaction-details-fetched"
 }
 /**
  * UFO types of components been instrumented
  */
 export declare enum ComponentName {
     PICKER_RENDERED = "reactions-picker",
-    REACTIONS = "reactions-list"
+    REACTIONS = "reactions-list",
+    REACTION_ITEM = "reaction-item"
 }
 /**
  * Experience when the emoji picker is opened
@@ -27,6 +29,10 @@ export declare const ReactionsRendered: ConcurrentExperience;
  * Experience when a reaction emoji gets added
  */
 export declare const ReactionsAdd: ConcurrentExperience;
+/**
+ * Experience when a reaction details gets fetched
+ */
+export declare const ReactionDetailsFetch: ConcurrentExperience;
 /**
  * Experience when a reaction emoji gets removed/decrement
  */

package/dist/types/client/ReactionClient.d.ts

@@ -4,8 +4,22 @@ export declare type ReactionRequest<T> = (containerAri: string, ari: string, emo
     [k: string]: any;
 }) => Promise<T>;
 export interface ReactionClient {
+    /**
+     * Fetch list of reactions for a given ARI.
+     * @param containerAri F
+     * @param aris
+     */
     getReactions(containerAri: string, aris: string[]): Promise<Reactions>;
+    /**
+     * Fetch details for a given reaction.
+     */
     getDetailedReaction: ReactionRequest<ReactionSummary>;
+    /**
+     * Fetch request when adding a reaction to a container.
+     */
     addReaction: ReactionRequest<ReactionSummary[]>;
+    /**
+     * Fetch request when removing a reaction from a container.
+     */
     deleteReaction: ReactionRequest<ReactionSummary[]>;
 }

package/dist/types/components/ReactionTooltip.d.ts

@@ -1,3 +1,4 @@
+/** @jsx jsx */
 import React from 'react';
 import { ReactionSummary } from '../types/ReactionSummary';
 export interface Props {

package/dist/types/components/UfoErrorBoundary.d.ts

@@ -1,8 +1,8 @@
-import React from 'react';
+import React, { ErrorInfo } from 'react';
 import { UFOExperience } from '@atlaskit/ufo';
 export declare class UfoErrorBoundary extends React.Component<{
     experiences: UFOExperience[];
 }> {
-    componentDidCatch(): void;
+    componentDidCatch(error: Error, errorInfo: ErrorInfo): void;
     render(): React.ReactNode;
 }

package/dist/types/store/ReactionsStore.d.ts

@@ -20,9 +20,22 @@ export interface StoreMetadata {
  * Set of all available UFO experiences relating to reaction element
  */
 export declare const ufoExperiences: {
+    /**
+     * Experience when a reaction emoji gets added
+     */
     add: import("@atlaskit/ufo").ConcurrentExperience;
+    /**
+     * Experience when a reaction emoji gets removed/decrement
+     */
     remove: import("@atlaskit/ufo").ConcurrentExperience;
+    /**
+     * Experience when the list of reactions gets rendered
+     */
     render: import("@atlaskit/ufo").ConcurrentExperience;
+    /**
+     * Experience when a reaction details gets fetched
+     */
+    fetchDetails: import("@atlaskit/ufo").ConcurrentExperience;
 };
 export interface ReactionsStore extends Types.Actions {
     getState: () => State;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@atlaskit/reactions",
-  "version": "21.3.1",
+  "version": "21.3.4",
   "description": "Reactions component",
   "publishConfig": {
     "registry": "https://registry.npmjs.org/"
@@ -29,10 +29,10 @@
     "@atlaskit/analytics-namespaced-context": "^6.4.0",
     "@atlaskit/analytics-next": "^8.2.0",
     "@atlaskit/button": "^16.3.0",
-    "@atlaskit/ds-lib": "^1.4.0",
-    "@atlaskit/emoji": "^64.6.0",
+    "@atlaskit/ds-lib": "^2.0.0",
+    "@atlaskit/emoji": "^64.7.0",
     "@atlaskit/icon": "^21.10.0",
-    "@atlaskit/motion": "^1.0.3",
+    "@atlaskit/motion": "^1.1.0",
     "@atlaskit/popper": "^5.0.0",
     "@atlaskit/theme": "^12.1.0",
     "@atlaskit/tokens": "^0.10.0",
@@ -53,7 +53,7 @@
     "@atlaskit/editor-test-helpers": "^17.0.0",
     "@atlaskit/elements-test-helpers": "^0.7.0",
     "@atlaskit/tooltip": "^17.5.0",
-    "@atlaskit/util-data-test": "^17.2.0",
+    "@atlaskit/util-data-test": "^17.3.0",
     "@atlassian/atlassian-frontend-prettier-config-1.0.1": "npm:@atlassian/atlassian-frontend-prettier-config@1.0.1",
     "@atlassian/ufo": "^0.1.0",
     "@types/webpack-dev-server": "^3.11.2",