@atlaskit/smart-card 38.8.0 → 38.9.1

package/CHANGELOG.md

@@ -1,5 +1,23 @@
 # @atlaskit/smart-card
 
+## 38.9.1
+
+### Patch Changes
+
+- [#167423](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/pull-requests/167423)
+  [`064579ff3e75c`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/064579ff3e75c) -
+  Move smart_links_noun_support feature gate to link-extractor
+- Updated dependencies
+
+## 38.9.0
+
+### Minor Changes
+
+- [#166270](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/pull-requests/166270)
+  [`231ec9798652d`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/231ec9798652d) -
+  [ux] Added hideIconLoadingSkeleton prop to CardSSR to allow for hiding the loading skeleton for
+  image icons in the title of block cards. Set to true in the renderer.
+
 ## 38.8.0
 
 ### Minor Changes

package/dist/cjs/extractors/common/context/index.js

@@ -14,6 +14,7 @@ exports.extractSmartLinkContext = extractSmartLinkContext;
 exports.generateContext = generateContext;
 var _defineProperty2 = _interopRequireDefault(require("@babel/runtime/helpers/defineProperty"));
 var _linkExtractors = require("@atlaskit/link-extractors");
+var _jsonld = require("../../../utils/jsonld");
 var _prioritiseIcon = require("../icon/prioritiseIcon");
 var _extractAccessContext = require("./extractAccessContext");
 function ownKeys(e, r) { var t = Object.keys(e); if (Object.getOwnPropertySymbols) { var o = Object.getOwnPropertySymbols(e); r && (o = o.filter(function (r) { return Object.getOwnPropertyDescriptor(e, r).enumerable; })), t.push.apply(t, o); } return t; }
@@ -48,5 +49,5 @@ function extractSmartLinkContext(response) {
   if ((0, _linkExtractors.isEntityPresent)(response)) {
     return (0, _linkExtractors.extractEntityProvider)(response);
   }
-  return generateContext(response === null || response === void 0 ? void 0 : response.data);
+  return generateContext((response === null || response === void 0 ? void 0 : response.data) || (0, _jsonld.getEmptyJsonLd)());
 }

package/dist/cjs/extractors/embed/index.js

@@ -25,23 +25,11 @@ var extractEmbedPreview = function extractEmbedPreview(jsonLd, platform, iframeU
 var extractEmbedProps = exports.extractEmbedProps = function extractEmbedProps(response, platform, iframeUrlType) {
   var meta = response === null || response === void 0 ? void 0 : response.meta;
   var jsonLd = (response === null || response === void 0 ? void 0 : response.data) || (0, _jsonld.getEmptyJsonLd)();
-  if ((0, _platformFeatureFlags.fg)('smart_links_noun_support')) {
-    return _objectSpread({
-      link: (0, _linkExtractors.extractSmartLinkUrl)(response) || '',
-      title: (0, _linkExtractors.extractSmartLinkTitle)(response),
-      context: (0, _context.extractSmartLinkContext)(response),
-      preview: (0, _linkExtractors.extractSmartLinkEmbed)(response, iframeUrlType),
-      isTrusted: (0, _extractIsTrusted.extractIsTrusted)(meta),
-      isSupportTheming: (0, _extractIsSupportTheming.extractIsSupportTheming)(meta)
-    }, (0, _platformFeatureFlags.fg)('platform-linking-visual-refresh-v2') && {
-      type: (0, _linkExtractors.extractType)(jsonLd)
-    });
-  }
   return _objectSpread({
-    link: (0, _linkExtractors.extractLink)(jsonLd) || '',
-    title: (0, _linkExtractors.extractTitle)(jsonLd),
-    context: (0, _context.generateContext)(jsonLd),
-    preview: extractEmbedPreview(jsonLd, platform, iframeUrlType),
+    link: (0, _linkExtractors.extractSmartLinkUrl)(response) || '',
+    title: (0, _linkExtractors.extractSmartLinkTitle)(response),
+    context: (0, _context.extractSmartLinkContext)(response),
+    preview: (0, _platformFeatureFlags.fg)('smart_links_noun_support') ? (0, _linkExtractors.extractSmartLinkEmbed)(response, iframeUrlType) : extractEmbedPreview(jsonLd, platform, iframeUrlType),
     isTrusted: (0, _extractIsTrusted.extractIsTrusted)(meta),
     isSupportTheming: (0, _extractIsSupportTheming.extractIsSupportTheming)(meta)
   }, (0, _platformFeatureFlags.fg)('platform-linking-visual-refresh-v2') && {

package/dist/cjs/extractors/flexible/extract-link-title.js

@@ -5,14 +5,12 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.default = void 0;
 var _linkExtractors = require("@atlaskit/link-extractors");
-var _platformFeatureFlags = require("@atlaskit/platform-feature-flags");
 var _constants = require("../../constants");
 var extractLinkTitle = function extractLinkTitle(status, propUrl, response, onClick) {
   var _ref;
-  var data = response === null || response === void 0 ? void 0 : response.data;
-  var responseUrl = (0, _platformFeatureFlags.fg)('smart_links_noun_support') ? (0, _linkExtractors.extractSmartLinkUrl)(response) : (0, _linkExtractors.extractLink)(data);
+  var responseUrl = (0, _linkExtractors.extractSmartLinkUrl)(response);
   var url = status === _constants.SmartLinkStatus.Resolved ? responseUrl !== null && responseUrl !== void 0 ? responseUrl : propUrl : propUrl;
-  var name = (0, _platformFeatureFlags.fg)('smart_links_noun_support') ? (0, _linkExtractors.extractSmartLinkTitle)(response) : (0, _linkExtractors.extractTitle)(data);
+  var name = (0, _linkExtractors.extractSmartLinkTitle)(response);
   var text = status === _constants.SmartLinkStatus.Resolved ? (_ref = name !== null && name !== void 0 ? name : responseUrl) !== null && _ref !== void 0 ? _ref : propUrl : propUrl;
   return {
     onClick: onClick,

package/dist/cjs/extractors/flexible/index.js

@@ -1,7 +1,6 @@
 "use strict";
 
 var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefault");
-var _typeof = require("@babel/runtime/helpers/typeof");
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
@@ -14,15 +13,14 @@ var _primitives = require("../common/primitives");
 var _actions = require("./actions");
 var _collaboratorGroup = require("./collaboratorGroup");
 var _extractLinkTitle = _interopRequireDefault(require("./extract-link-title"));
-var _extractPreview = _interopRequireWildcard(require("./extract-preview"));
+var _extractPreview = require("./extract-preview");
 var _extractPriority = _interopRequireDefault(require("./extract-priority"));
 var _extractState = _interopRequireDefault(require("./extract-state"));
 var _icon = require("./icon");
-var _extractProviderIcon = _interopRequireWildcard(require("./icon/extract-provider-icon"));
+var _extractProviderIcon = require("./icon/extract-provider-icon");
 var _latestCommit = require("./latest-commit");
 var _utils = require("./utils");
 var _excluded = ["appearance", "fireEvent", "id", "onClick", "origin", "renderers", "resolve", "actionOptions", "response", "status", "aiSummaryConfig"];
-function _interopRequireWildcard(e, t) { if ("function" == typeof WeakMap) var r = new WeakMap(), n = new WeakMap(); return (_interopRequireWildcard = function _interopRequireWildcard(e, t) { if (!t && e && e.__esModule) return e; var o, i, f = { __proto__: null, default: e }; if (null === e || "object" != _typeof(e) && "function" != typeof e) return f; if (o = t ? n : r) { if (o.has(e)) return o.get(e); o.set(e, f); } for (var _t in e) "default" !== _t && {}.hasOwnProperty.call(e, _t) && ((i = (o = Object.defineProperty) && Object.getOwnPropertyDescriptor(e, _t)) && (i.get || i.set) ? o(f, _t, i) : f[_t] = e[_t]); return f; })(e, t); }
 function ownKeys(e, r) { var t = Object.keys(e); if (Object.getOwnPropertySymbols) { var o = Object.getOwnPropertySymbols(e); r && (o = o.filter(function (r) { return Object.getOwnPropertyDescriptor(e, r).enumerable; })), t.push.apply(t, o); } return t; }
 function _objectSpread(e) { for (var r = 1; r < arguments.length; r++) { var t = null != arguments[r] ? arguments[r] : {}; r % 2 ? ownKeys(Object(t), !0).forEach(function (r) { (0, _defineProperty2.default)(e, r, t[r]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(e, Object.getOwnPropertyDescriptors(t)) : ownKeys(Object(t)).forEach(function (r) { Object.defineProperty(e, r, Object.getOwnPropertyDescriptor(t, r)); }); } return e; }
 var extractFlexibleUiContext = function extractFlexibleUiContext() {
@@ -44,7 +42,7 @@ var extractFlexibleUiContext = function extractFlexibleUiContext() {
   }
   var data = response.data;
   var meta = response.meta;
-  var url = (0, _platformFeatureFlags.fg)('smart_links_noun_support') ? (0, _linkExtractors.extractSmartLinkUrl)(response) : (0, _linkExtractors.extractLink)(data);
+  var url = (0, _linkExtractors.extractSmartLinkUrl)(response);
   return _objectSpread(_objectSpread(_objectSpread(_objectSpread({
     actions: (0, _actions.extractFlexibleCardActions)({
       actionOptions: actionOptions,
@@ -67,22 +65,22 @@ var extractFlexibleUiContext = function extractFlexibleUiContext() {
     reactCount: (0, _utils.extractReactCount)(data),
     voteCount: (0, _utils.extractVoteCount)(data),
     checklistProgress: (0, _utils.extractChecklistProgress)(data),
-    createdBy: (0, _platformFeatureFlags.fg)('smart_links_noun_support') ? (0, _linkExtractors.extractSmartLinkCreatedBy)(response) : (0, _utils.extractCreatedBy)(data),
+    createdBy: (0, _linkExtractors.extractSmartLinkCreatedBy)(response),
     ownedBy: (0, _utils.extractOwnedBy)(data),
     assignedTo: (0, _utils.extractAssignedTo)(data),
-    createdOn: (0, _platformFeatureFlags.fg)('smart_links_noun_support') ? (0, _linkExtractors.extractSmartLinkCreatedOn)(response) : (0, _linkExtractors.extractDateCreated)(data),
+    createdOn: (0, _linkExtractors.extractSmartLinkCreatedOn)(response),
     dueOn: (0, _utils.extractDueOn)(data),
     latestCommit: (0, _latestCommit.extractLatestCommit)(data),
-    linkIcon: (0, _platformFeatureFlags.fg)('smart_links_noun_support') ? (0, _icon.extractSmartLinkIcon)(response) : (0, _icon.extractLinkIcon)(response, renderers)
+    linkIcon: (0, _icon.extractSmartLinkIcon)(response)
   }, (0, _platformFeatureFlags.fg)('platform-linking-flexible-card-context') && {
     linkTitle: (0, _extractLinkTitle.default)(status, props.url, response, onClick)
   }), {}, {
     location: (0, _utils.extractLocation)(data),
-    modifiedBy: (0, _platformFeatureFlags.fg)('smart_links_noun_support') ? (0, _linkExtractors.extractSmartLinkModifiedBy)(response) : (0, _utils.extractModifiedBy)(data),
-    modifiedOn: (0, _platformFeatureFlags.fg)('smart_links_noun_support') ? (0, _linkExtractors.extractSmartLinkModifiedOn)(response) : (0, _linkExtractors.extractDateUpdated)(data),
-    preview: (0, _platformFeatureFlags.fg)('smart_links_noun_support') ? (0, _extractPreview.extractSmartLinkPreviewImage)(response) : (0, _extractPreview.default)(data),
+    modifiedBy: (0, _linkExtractors.extractSmartLinkModifiedBy)(response),
+    modifiedOn: (0, _linkExtractors.extractSmartLinkModifiedOn)(response),
+    preview: (0, _extractPreview.extractSmartLinkPreviewImage)(response),
     priority: (0, _extractPriority.default)(data),
-    provider: (0, _platformFeatureFlags.fg)('smart_links_noun_support') ? (0, _extractProviderIcon.extractSmartLinkProviderIcon)(response) : (0, _extractProviderIcon.default)(data),
+    provider: (0, _extractProviderIcon.extractSmartLinkProviderIcon)(response),
     programmingLanguage: (0, _utils.extractProgrammingLanguage)(data),
     readTime: (0, _utils.extractReadTime)(data),
     sentOn: (0, _utils.extractSentOn)(data),
@@ -95,10 +93,10 @@ var extractFlexibleUiContext = function extractFlexibleUiContext() {
     storyPoints: (0, _utils.extractStoryPoints)(data),
     targetBranch: (0, _utils.extractTargetBranch)(data)
   }, (0, _platformFeatureFlags.fg)('platform-linking-flexible-card-context') ? undefined : {
-    title: (0, _platformFeatureFlags.fg)('smart_links_noun_support') ? (0, _linkExtractors.extractSmartLinkTitle)(response) || url : (0, _linkExtractors.extractTitle)(data) || url
+    title: (0, _linkExtractors.extractSmartLinkTitle)(response) || url
   }), {}, {
     url: url,
-    ari: (0, _platformFeatureFlags.fg)('smart_links_noun_support') ? (0, _linkExtractors.extractSmartLinkAri)(response) : (0, _linkExtractors.extractAri)(data)
+    ari: (0, _linkExtractors.extractSmartLinkAri)(response)
   }, (0, _platformFeatureFlags.fg)('platform-linking-visual-refresh-v2') && {
     type: (0, _linkExtractors.extractType)(data)
   }), (0, _platformFeatureFlags.fg)('cc-ai-linking-platform-snippet-renderer') && {

package/dist/cjs/extractors/inline/index.js

@@ -8,6 +8,7 @@ exports.extractInlineProps = void 0;
 var _defineProperty2 = _interopRequireDefault(require("@babel/runtime/helpers/defineProperty"));
 var _linkExtractors = require("@atlaskit/link-extractors");
 var _platformFeatureFlags = require("@atlaskit/platform-feature-flags");
+var _jsonld = require("../../utils/jsonld");
 var _icon = require("../common/icon");
 var _lozenge = require("../common/lozenge");
 var _primitives = require("../common/primitives");
@@ -42,29 +43,17 @@ var extractSmartLinkInlineIcon = function extractSmartLinkInlineIcon(response) {
     // see: https://product-fabric.atlassian.net/browse/EDM-12375
     // return extractSmartLinkIcon(response);
   }
-  return extractInlineIcon(response === null || response === void 0 ? void 0 : response.data, showLabel);
+  return extractInlineIcon((response === null || response === void 0 ? void 0 : response.data) || (0, _jsonld.getEmptyJsonLd)(), showLabel);
 };
 var extractInlineProps = exports.extractInlineProps = function extractInlineProps(response, renderers, removeTextHighlightingFromTitle) {
   var showLabel = arguments.length > 3 && arguments[3] !== undefined ? arguments[3] : true;
-  var jsonLd = response === null || response === void 0 ? void 0 : response.data;
-  if ((0, _platformFeatureFlags.fg)('smart_links_noun_support')) {
-    return _objectSpread({
-      link: (0, _linkExtractors.extractSmartLinkUrl)(response),
-      title: (0, _linkExtractors.extractSmartLinkTitle)(response, removeTextHighlightingFromTitle),
-      icon: extractSmartLinkInlineIcon(response, showLabel),
-      // As we migrate to support more entities we can incorporate these fields
-      lozenge: (0, _lozenge.extractLozenge)(jsonLd),
-      titleTextColor: (0, _primitives.extractTitleTextColor)(jsonLd),
-      titlePrefix: (0, _titlePrefix.extractTitlePrefix)(jsonLd, renderers, 'inline')
-    }, (0, _platformFeatureFlags.fg)('platform-linking-visual-refresh-v2') && {
-      type: (0, _linkExtractors.extractType)(jsonLd)
-    });
-  }
+  var jsonLd = (response === null || response === void 0 ? void 0 : response.data) || (0, _jsonld.getEmptyJsonLd)();
   return _objectSpread({
-    link: (0, _linkExtractors.extractLink)(jsonLd),
-    title: (0, _linkExtractors.extractTitle)(jsonLd, removeTextHighlightingFromTitle),
+    link: (0, _linkExtractors.extractSmartLinkUrl)(response),
+    title: (0, _linkExtractors.extractSmartLinkTitle)(response, removeTextHighlightingFromTitle),
+    icon: extractSmartLinkInlineIcon(response, showLabel),
+    // As we migrate to support more entities we can incorporate these fields
     lozenge: (0, _lozenge.extractLozenge)(jsonLd),
-    icon: extractInlineIcon(jsonLd, showLabel),
     titleTextColor: (0, _primitives.extractTitleTextColor)(jsonLd),
     titlePrefix: (0, _titlePrefix.extractTitlePrefix)(jsonLd, renderers, 'inline')
   }, (0, _platformFeatureFlags.fg)('platform-linking-visual-refresh-v2') && {

package/dist/cjs/state/hooks/use-resolve/index.js

@@ -10,7 +10,6 @@ var _asyncToGenerator2 = _interopRequireDefault(require("@babel/runtime/helpers/
 var _react = require("react");
 var _linkExtractors = require("@atlaskit/link-extractors");
 var _linkProvider = require("@atlaskit/link-provider");
-var _platformFeatureFlags = require("@atlaskit/platform-feature-flags");
 var _constants = require("../../../constants");
 var _analytics = require("../../analytics");
 var _useResponse2 = _interopRequireDefault(require("../use-response"));
@@ -45,7 +44,7 @@ var useResolve = function useResolve() {
               status: _constants.SmartLinkStatus.Pending,
               details: undefined
             }, details = _ref2.details;
-            hasData = (0, _platformFeatureFlags.fg)('smart_links_noun_support') ? !!(details && details.data || (0, _linkExtractors.isEntityPresent)(details)) : !!(details && details.data);
+            hasData = !!(details && details.data || (0, _linkExtractors.isEntityPresent)(details));
             if (!(isReloading || !hasData || isMetadataRequest)) {
               _context.next = 9;
               break;

package/dist/cjs/utils/analytics/analytics.js

@@ -11,7 +11,7 @@ var ANALYTICS_CHANNEL = exports.ANALYTICS_CHANNEL = 'media';
 var context = exports.context = {
   componentName: 'smart-cards',
   packageName: "@atlaskit/smart-card",
-  packageVersion: "38.8.0"
+  packageVersion: "38.9.1"
 };
 var TrackQuickActionType = exports.TrackQuickActionType = /*#__PURE__*/function (TrackQuickActionType) {
   TrackQuickActionType["StatusUpdate"] = "StatusUpdate";

package/dist/cjs/view/BlockCard/index.js

@@ -30,7 +30,8 @@ var BlockCard = exports.BlockCard = function BlockCard(_ref) {
     onError = _ref.onError,
     testId = _ref.testId,
     actionOptions = _ref.actionOptions,
-    CompetitorPrompt = _ref.CompetitorPrompt;
+    CompetitorPrompt = _ref.CompetitorPrompt,
+    hideIconLoadingSkeleton = _ref.hideIconLoadingSkeleton;
   var status = cardState.status;
   var blockCardProps = {
     id: id,
@@ -44,7 +45,8 @@ var BlockCard = exports.BlockCard = function BlockCard(_ref) {
     onResolve: onResolve,
     renderers: renderers,
     actionOptions: actionOptions,
-    CompetitorPrompt: CompetitorPrompt
+    CompetitorPrompt: CompetitorPrompt,
+    hideIconLoadingSkeleton: hideIconLoadingSkeleton
   };
   var _useControlDataExport = (0, _useControlDataExportConfig.useControlDataExportConfig)(),
     _useControlDataExport2 = _useControlDataExport.shouldControlDataExport,

package/dist/cjs/view/BlockCard/views/ForbiddenView.js

@@ -53,7 +53,7 @@ var ForbiddenView = function ForbiddenView(_ref) {
     url = props.url;
   var details = cardState === null || cardState === void 0 ? void 0 : cardState.details;
   var cardMetadata = (_details$meta = details === null || details === void 0 ? void 0 : details.meta) !== null && _details$meta !== void 0 ? _details$meta : (0, _jsonld.getForbiddenJsonLd)().meta;
-  var provider = (0, _platformFeatureFlags.fg)('smart_links_noun_support') ? (0, _linkExtractors.extractSmartLinkProvider)(details) : (0, _linkExtractors.extractProvider)(details === null || details === void 0 ? void 0 : details.data);
+  var provider = (0, _linkExtractors.extractSmartLinkProvider)(details);
   var providerName = (provider === null || provider === void 0 ? void 0 : provider.text) || '';
   var messageContext = (0, _react.useMemo)(function () {
     var hostname = /*#__PURE__*/React.createElement("b", null, (0, _extractHostname.default)(url));

package/dist/cjs/view/BlockCard/views/NotFoundView.js

@@ -40,7 +40,7 @@ var NotFoundView = function NotFoundView(_ref) {
   var details = props.cardState.details;
   var product = (0, _react.useMemo)(function () {
     var _provider$text;
-    var provider = (0, _platformFeatureFlags.fg)('smart_links_noun_support') ? (0, _linkExtractors.extractSmartLinkProvider)(details) : (0, _linkExtractors.extractProvider)(details === null || details === void 0 ? void 0 : details.data);
+    var provider = (0, _linkExtractors.extractSmartLinkProvider)(details);
     return (_provider$text = provider === null || provider === void 0 ? void 0 : provider.text) !== null && _provider$text !== void 0 ? _provider$text : '';
   }, [details]);
   var title = (0, _react.useMemo)(function () {

package/dist/cjs/view/BlockCard/views/ResolvedView.js

@@ -42,7 +42,8 @@ var ResolvedView = function ResolvedView(_ref) {
     _ref$testId = _ref.testId,
     testId = _ref$testId === void 0 ? 'smart-block-resolved-view' : _ref$testId,
     url = _ref.url,
-    CompetitorPrompt = _ref.CompetitorPrompt;
+    CompetitorPrompt = _ref.CompetitorPrompt,
+    hideIconLoadingSkeleton = _ref.hideIconLoadingSkeleton;
   var _useState = (0, _react.useState)(false),
     _useState2 = (0, _slicedToArray2.default)(_useState, 2),
     isPreviewBlockErrored = _useState2[0],
@@ -100,6 +101,7 @@ var ResolvedView = function ResolvedView(_ref) {
   }, {
     CompetitorPrompt: CompetitorPrompt,
     url: url,
+    hideIconLoadingSkeleton: hideIconLoadingSkeleton,
     className: (0, _runtime.ax)([(0, _platformFeatureFlags.fg)('platform-linking-visual-refresh-v1') ? "_zulpu2gc _13mh1pd9 _2yhz14q2" : "_zulpcxkx _13mh1pd9"])
   })), /*#__PURE__*/React.createElement(_blocks.MetadataBlock, (0, _extends2.default)({
     primary: topMetadata,

package/dist/cjs/view/BlockCard/views/UnauthorisedView.js

@@ -36,14 +36,13 @@ var contentStyles = null;
  * @see FlexibleCardProps
  */
 var UnauthorisedView = function UnauthorisedView(_ref) {
-  var _cardState$details, _extractSmartLinkProv, _extractProvider;
+  var _extractSmartLinkProv;
   var _ref$testId = _ref.testId,
     testId = _ref$testId === void 0 ? 'smart-block-unauthorized-view' : _ref$testId,
     props = (0, _objectWithoutProperties2.default)(_ref, _excluded);
   var cardState = props.cardState,
     onAuthorize = props.onAuthorize;
-  var data = (_cardState$details = cardState.details) === null || _cardState$details === void 0 ? void 0 : _cardState$details.data;
-  var providerName = (0, _platformFeatureFlags.fg)('smart_links_noun_support') ? (_extractSmartLinkProv = (0, _linkExtractors.extractSmartLinkProvider)(cardState.details)) === null || _extractSmartLinkProv === void 0 ? void 0 : _extractSmartLinkProv.text : (_extractProvider = (0, _linkExtractors.extractProvider)(data)) === null || _extractProvider === void 0 ? void 0 : _extractProvider.text;
+  var providerName = (_extractSmartLinkProv = (0, _linkExtractors.extractSmartLinkProvider)(cardState.details)) === null || _extractSmartLinkProv === void 0 ? void 0 : _extractSmartLinkProv.text;
   var isProductIntegrationSupported = (0, _helpers.hasAuthScopeOverrides)(cardState.details);
   var _useAnalyticsEvents = (0, _useAnalyticsEvents2.useAnalyticsEvents)(),
     fireEvent = _useAnalyticsEvents.fireEvent;

package/dist/cjs/view/CardWithUrl/component.js

@@ -50,7 +50,8 @@ function Component(_ref) {
     removeTextHighlightingFromTitle = _ref.removeTextHighlightingFromTitle,
     resolvingPlaceholder = _ref.resolvingPlaceholder,
     truncateInline = _ref.truncateInline,
-    CompetitorPrompt = _ref.CompetitorPrompt;
+    CompetitorPrompt = _ref.CompetitorPrompt,
+    hideIconLoadingSkeleton = _ref.hideIconLoadingSkeleton;
   var _useAnalyticsEventsNe = (0, _analyticsNext.useAnalyticsEvents)(),
     createAnalyticsEvent = _useAnalyticsEventsNe.createAnalyticsEvent;
   var _useAnalyticsEvents = (0, _useAnalyticsEvents2.useAnalyticsEvents)(),
@@ -80,12 +81,7 @@ function Component(_ref) {
 
   // Setup UI handlers.
   var handleClickWrapper = (0, _react.useCallback)(function (event) {
-    var isModifierKeyPressed = (0, _utils.isSpecialEvent)(event);
-    // Ctrl+left click on mac typically doesn't trigger onClick
-    // The event could have potentially had `e.preventDefault()` called on it by now
-    // event by smart card internally
-    // If it has been called then only then can `isModifierKeyPressed` be true.
-    var target = isModifierKeyPressed ? '_blank' : '_self';
+    var isModifierKeyPressed = (0, _utils.isSpecialKey)(event) || (0, _utils.isSpecialClick)(event);
     fireEvent('ui.smartLink.clicked', {
       id: id,
       display: isFlexibleUi ? _constants.CardDisplay.Flexible : appearance,
@@ -97,9 +93,11 @@ function Component(_ref) {
     // delegate the click to the glance panel handler
     if (
     // eslint-disable-next-line @atlaskit/platform/no-preconditioning
-    (0, _platformFeatureFlags.fg)('fun-1765_wire_up_glance_panel_to_smart_cards') && target !== '_blank' && ari && isGlancePanelAvailable !== null && isGlancePanelAvailable !== void 0 && isGlancePanelAvailable({
+    (0, _platformFeatureFlags.fg)('fun-1765_wire_up_glance_panel_to_smart_cards') && !isModifierKeyPressed && ari && openGlancePanel && isGlancePanelAvailable !== null && isGlancePanelAvailable !== void 0 && isGlancePanelAvailable({
       ari: ari
-    }) && openGlancePanel) {
+    })) {
+      event.preventDefault();
+      event.stopPropagation();
       openGlancePanel({
         url: url,
         ari: ari,
@@ -113,6 +111,12 @@ function Component(_ref) {
       });
     } else if (!onClick && !isFlexibleUi) {
       var clickUrl = (0, _helpers.getClickUrl)(url, state.details);
+
+      // Ctrl+left click on mac typically doesn't trigger onClick
+      // The event could have potentially had `e.preventDefault()` called on it by now
+      // event by smart card internally
+      // If it has been called then only then can `isSpecialEvent` be true.
+      var target = (0, _utils.isSpecialEvent)(event) ? '_blank' : '_self';
       window.open(clickUrl, target);
       (0, _click.fireLinkClickedEvent)(createAnalyticsEvent)(event, {
         attributes: {
@@ -268,7 +272,8 @@ function Component(_ref) {
         onError: onError,
         testId: testId,
         actionOptions: actionOptions,
-        CompetitorPrompt: CompetitorPrompt
+        CompetitorPrompt: CompetitorPrompt,
+        hideIconLoadingSkeleton: hideIconLoadingSkeleton
       });
     case 'embed':
       return /*#__PURE__*/_react.default.createElement(_EmbedCard.EmbedCard, {

package/dist/cjs/view/FlexibleCard/components/blocks/action-block/action-footer/index.js

@@ -10,7 +10,7 @@ exports.ActionFooter = void 0;
 require("./index.compiled.css");
 var React = _interopRequireWildcard(require("react"));
 var _runtime = require("@compiled/react/runtime");
-var _error = _interopRequireDefault(require("@atlaskit/icon/utility/migration/error"));
+var _error = _interopRequireDefault(require("@atlaskit/icon/core/migration/error"));
 var _platformFeatureFlags = require("@atlaskit/platform-feature-flags");
 var _compiled = require("@atlaskit/primitives/compiled");
 var _motionWrapper = _interopRequireDefault(require("../../../common/motion-wrapper"));
@@ -31,7 +31,8 @@ var getIcon = function getIcon(appearance) {
         label: "Error",
         LEGACY_size: "small",
         testId: "action-error-icon",
-        LEGACY_margin: "0 ".concat("var(--ds-space-negative-025, -2px)")
+        LEGACY_margin: "0 ".concat("var(--ds-space-negative-025, -2px)"),
+        size: "small"
       });
     default:
       return null;

package/dist/cjs/view/FlexibleCard/components/blocks/title-block/index.js

@@ -23,7 +23,7 @@ var _actionGroup = _interopRequireDefault(require("../action-group"));
 var _errored = _interopRequireDefault(require("./errored"));
 var _resolved = _interopRequireDefault(require("./resolved"));
 var _resolving = _interopRequireDefault(require("./resolving"));
-var _excluded = ["actions", "anchorTarget", "hideTitleTooltip", "maxLines", "onActionMenuOpenChange", "onClick", "status", "showActionOnHover", "testId", "text", "icon", "theme", "hideRetry", "metadataPosition", "hideIcon", "className", "anchorRef", "CompetitorPrompt", "url"];
+var _excluded = ["actions", "anchorTarget", "hideTitleTooltip", "maxLines", "onActionMenuOpenChange", "onClick", "status", "showActionOnHover", "testId", "text", "icon", "theme", "hideRetry", "metadataPosition", "hideIcon", "className", "anchorRef", "CompetitorPrompt", "url", "hideIconLoadingSkeleton"];
 function _interopRequireWildcard(e, t) { if ("function" == typeof WeakMap) var r = new WeakMap(), n = new WeakMap(); return (_interopRequireWildcard = function _interopRequireWildcard(e, t) { if (!t && e && e.__esModule) return e; var o, i, f = { __proto__: null, default: e }; if (null === e || "object" != _typeof(e) && "function" != typeof e) return f; if (o = t ? n : r) { if (o.has(e)) return o.get(e); o.set(e, f); } for (var _t in e) "default" !== _t && {}.hasOwnProperty.call(e, _t) && ((i = (o = Object.defineProperty) && Object.getOwnPropertyDescriptor(e, _t)) && (i.get || i.set) ? o(f, _t, i) : f[_t] = e[_t]); return f; })(e, t); }
 var actionStyles = null;
 var getTitleBlockViewComponent = function getTitleBlockViewComponent(status) {
@@ -80,6 +80,7 @@ var TitleBlock = function TitleBlock(_ref) {
     anchorRef = _ref.anchorRef,
     CompetitorPrompt = _ref.CompetitorPrompt,
     url = _ref.url,
+    hideIconLoadingSkeleton = _ref.hideIconLoadingSkeleton,
     props = (0, _objectWithoutProperties2.default)(_ref, _excluded);
   var cardContext = (0, _platformFeatureFlags.fg)('platform-linking-flexible-card-context') ?
   // eslint-disable-next-line react-hooks/rules-of-hooks
@@ -139,6 +140,7 @@ var TitleBlock = function TitleBlock(_ref) {
     hideRetry: hideRetry
   } : undefined, {
     icon: icon,
+    hideIconLoadingSkeleton: hideIconLoadingSkeleton,
     size: (0, _platformFeatureFlags.fg)('platform-linking-flexible-card-context') ? (_props$size = props.size) !== null && _props$size !== void 0 ? _props$size : ui === null || ui === void 0 ? void 0 : ui.size : props === null || props === void 0 ? void 0 : props.size
   }, (0, _platformFeatureFlags.fg)('platform-linking-flexible-card-context') ? undefined : {
     theme: theme

package/dist/cjs/view/FlexibleCard/components/blocks/title-block/resolved/index.js

@@ -19,7 +19,7 @@ var _elements = require("../../../elements");
 var _block = _interopRequireDefault(require("../../block"));
 var _elementGroup = _interopRequireDefault(require("../../element-group"));
 var _utils = require("../../utils");
-var _excluded = ["actionGroup", "metadata", "position", "subtitle", "testId", "text", "icon", "title", "metadataPosition", "hideIcon", "CompetitorPrompt", "url"];
+var _excluded = ["actionGroup", "metadata", "position", "subtitle", "testId", "text", "icon", "title", "metadataPosition", "hideIcon", "CompetitorPrompt", "url", "hideIconLoadingSkeleton"];
 function _interopRequireWildcard(e, t) { if ("function" == typeof WeakMap) var r = new WeakMap(), n = new WeakMap(); return (_interopRequireWildcard = function _interopRequireWildcard(e, t) { if (!t && e && e.__esModule) return e; var o, i, f = { __proto__: null, default: e }; if (null === e || "object" != _typeof(e) && "function" != typeof e) return f; if (o = t ? n : r) { if (o.has(e)) return o.get(e); o.set(e, f); } for (var _t in e) "default" !== _t && {}.hasOwnProperty.call(e, _t) && ((i = (o = Object.defineProperty) && Object.getOwnPropertyDescriptor(e, _t)) && (i.get || i.set) ? o(f, _t, i) : f[_t] = e[_t]); return f; })(e, t); }
 var style = null;
 var styles = {
@@ -46,6 +46,7 @@ var TitleBlockResolvedView = function TitleBlockResolvedView(_ref) {
     hideIcon = _ref.hideIcon,
     CompetitorPrompt = _ref.CompetitorPrompt,
     url = _ref.url,
+    hideIconLoadingSkeleton = _ref.hideIconLoadingSkeleton,
     blockProps = (0, _objectWithoutProperties2.default)(_ref, _excluded);
   var size = blockProps.size;
   var metadataElements = (0, _utils.renderElementItems)(metadata);
@@ -55,7 +56,8 @@ var TitleBlockResolvedView = function TitleBlockResolvedView(_ref) {
   }), !hideIcon && /*#__PURE__*/React.createElement(_elements.LinkIcon, {
     overrideIcon: icon,
     position: position,
-    size: size
+    size: size,
+    hideLoadingSkeleton: hideIconLoadingSkeleton
   }), /*#__PURE__*/React.createElement(_elementGroup.default, {
     direction: _constants.SmartLinkDirection.Vertical,
     width: _constants.SmartLinkWidth.Flexible,

package/dist/cjs/view/FlexibleCard/components/common/image-icon/index.js

@@ -8,8 +8,10 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.default = void 0;
 require("./index.compiled.css");
-var React = _interopRequireWildcard(require("react"));
+var _react = _interopRequireWildcard(require("react"));
+var React = _react;
 var _runtime = require("@compiled/react/runtime");
+var _slicedToArray2 = _interopRequireDefault(require("@babel/runtime/helpers/slicedToArray"));
 var _reactRenderImage = _interopRequireDefault(require("react-render-image"));
 var _platformFeatureFlags = require("@atlaskit/platform-feature-flags");
 var _loadingSkeleton = require("../loading-skeleton");
@@ -26,15 +28,23 @@ var ImageIcon = function ImageIcon(_ref) {
     _ref$appearance = _ref.appearance,
     appearance = _ref$appearance === void 0 ? 'square' : _ref$appearance,
     onError = _ref.onError,
-    onLoad = _ref.onLoad;
-  return /*#__PURE__*/React.createElement(_reactRenderImage.default, {
-    src: url,
-    loading: /*#__PURE__*/React.createElement(_loadingSkeleton.LoadingSkeleton, {
-      testId: "".concat(testId, "-loading"),
-      width: width,
-      height: height
-    }),
-    loaded: /*#__PURE__*/React.createElement("img", {
+    onLoad = _ref.onLoad,
+    _ref$hideLoadingSkele = _ref.hideLoadingSkeleton,
+    hideLoadingSkeleton = _ref$hideLoadingSkele === void 0 ? false : _ref$hideLoadingSkele;
+  var _useState = (0, _react.useState)(false),
+    _useState2 = (0, _slicedToArray2.default)(_useState, 2),
+    hasImageErrored = _useState2[0],
+    setHasImageErrored = _useState2[1];
+
+  // If url changes, reset state
+  (0, _react.useEffect)(function () {
+    setHasImageErrored(false);
+  }, [url]);
+  if (hasImageErrored) {
+    return defaultIcon;
+  }
+  if (hideLoadingSkeleton && (0, _platformFeatureFlags.fg)('platform_fix_block_card_img_icon_vc')) {
+    return /*#__PURE__*/React.createElement("img", {
       src: url,
       "data-testid": "".concat(testId, "-image"),
       alt: "",
@@ -42,11 +52,33 @@ var ImageIcon = function ImageIcon(_ref) {
         width: width,
         height: height
       },
+      onError: function onError() {
+        return setHasImageErrored(true);
+      },
       className: (0, _runtime.ax)([appearance === 'round' && (0, _platformFeatureFlags.fg)('platform-linking-visual-refresh-v2') && styles.roundImg])
-    }),
-    errored: defaultIcon,
-    onError: onError,
-    onLoad: onLoad
-  });
+    });
+  } else {
+    return /*#__PURE__*/React.createElement(_reactRenderImage.default, {
+      src: url,
+      loading: /*#__PURE__*/React.createElement(_loadingSkeleton.LoadingSkeleton, {
+        testId: "".concat(testId, "-loading"),
+        width: width,
+        height: height
+      }),
+      loaded: /*#__PURE__*/React.createElement("img", {
+        src: url,
+        "data-testid": "".concat(testId, "-image"),
+        alt: "",
+        style: {
+          width: width,
+          height: height
+        },
+        className: (0, _runtime.ax)([appearance === 'round' && (0, _platformFeatureFlags.fg)('platform-linking-visual-refresh-v2') && styles.roundImg])
+      }),
+      errored: defaultIcon,
+      onError: onError,
+      onLoad: onLoad
+    });
+  }
 };
 var _default = exports.default = ImageIcon;

package/dist/cjs/view/FlexibleCard/components/elements/common/base-icon-element/index.js

@@ -62,6 +62,7 @@ var renderDefaultIcon = function renderDefaultIcon(label, testId) {
 var renderImageIcon = function renderImageIcon(defaultIcon, url, testId) {
   var size = arguments.length > 3 && arguments[3] !== undefined ? arguments[3] : _constants.SmartLinkSize.Medium;
   var appearance = arguments.length > 4 ? arguments[4] : undefined;
+  var hideLoadingSkeleton = arguments.length > 5 ? arguments[5] : undefined;
   var width = size === _constants.SmartLinkSize.Large ? "var(--ds-space-300, 24px)" : "var(--ds-space-200, 16px)";
   if (url) {
     return /*#__PURE__*/React.createElement(_imageIcon.default, (0, _extends2.default)({
@@ -72,6 +73,8 @@ var renderImageIcon = function renderImageIcon(defaultIcon, url, testId) {
       height: width
     }, (0, _platformFeatureFlags.fg)('platform-linking-visual-refresh-v2') && {
       appearance: appearance
+    }, {
+      hideLoadingSkeleton: hideLoadingSkeleton
     }));
   }
 };
@@ -98,11 +101,12 @@ var IconElement = function IconElement(_ref) {
     testId = _ref$testId === void 0 ? 'smart-element-icon' : _ref$testId,
     url = _ref.url,
     _ref$appearance = _ref.appearance,
-    appearance = _ref$appearance === void 0 ? 'square' : _ref$appearance;
+    appearance = _ref$appearance === void 0 ? 'square' : _ref$appearance,
+    hideLoadingSkeleton = _ref.hideLoadingSkeleton;
   var element = (0, _react.useMemo)(function () {
     var defaultIcon = renderDefaultIcon(label, testId);
-    return overrideIcon || (render === null || render === void 0 ? void 0 : render()) || renderImageIcon(defaultIcon, url, testId, size, (0, _platformFeatureFlags.fg)('platform-linking-visual-refresh-v2') ? appearance : undefined) || renderAtlaskitIcon(icon, testId, size) || defaultIcon;
-  }, [label, testId, overrideIcon, render, url, size, appearance, icon]);
+    return overrideIcon || (render === null || render === void 0 ? void 0 : render()) || renderImageIcon(defaultIcon, url, testId, size, (0, _platformFeatureFlags.fg)('platform-linking-visual-refresh-v2') ? appearance : undefined, hideLoadingSkeleton) || renderAtlaskitIcon(icon, testId, size) || defaultIcon;
+  }, [label, testId, overrideIcon, render, url, size, appearance, hideLoadingSkeleton, icon]);
   var width = (0, _utils2.getIconWidth)(size);
   return /*#__PURE__*/React.createElement("div", (0, _extends2.default)({}, (0, _platformFeatureFlags.fg)('platform-linking-visual-refresh-v1') ? {} : (0, _defineProperty2.default)({}, 'data-fit-to-content', true), {
     "data-smart-element": name,

package/dist/cjs/view/FlexibleCard/components/elements/common/base-lozenge-element/lozenge-action/lozenge-action-trigger/index.js

@@ -15,7 +15,7 @@ var _extends2 = _interopRequireDefault(require("@babel/runtime/helpers/extends")
 var _slicedToArray2 = _interopRequireDefault(require("@babel/runtime/helpers/slicedToArray"));
 var _objectWithoutProperties2 = _interopRequireDefault(require("@babel/runtime/helpers/objectWithoutProperties"));
 var _react2 = require("@compiled/react");
-var _chevronDown = _interopRequireDefault(require("@atlaskit/icon/utility/migration/chevron-down"));
+var _chevronDown = _interopRequireDefault(require("@atlaskit/icon/core/migration/chevron-down"));
 var _linkingCommon = require("@atlaskit/linking-common");
 var _lozenge = _interopRequireDefault(require("@atlaskit/lozenge"));
 var _platformFeatureFlags = require("@atlaskit/platform-feature-flags");
@@ -61,7 +61,8 @@ var LozengeActionTriggerOld = function LozengeActionTriggerOld(_ref) {
       color: "currentColor",
       label: "options",
       LEGACY_size: "medium",
-      LEGACY_margin: "-4px -8px -4px -7px"
+      LEGACY_margin: "-4px -8px -4px -7px",
+      size: "small"
     }))));
   }, [appearance, isBold, text]);
   return /*#__PURE__*/React.createElement("button", (0, _extends2.default)({
@@ -151,7 +152,8 @@ var LozengeActionTrigger = function LozengeActionTrigger(_ref2) {
       color: "currentColor",
       label: "options",
       LEGACY_size: "medium",
-      LEGACY_margin: "-4px -8px -4px -7px"
+      LEGACY_margin: "-4px -8px -4px -7px",
+      size: "small"
     })))));
   }, [appearance, isHovering, isPressing, text, isOpen, lozengeBackgroundColor, lozengeForegroundColor]);
   return /*#__PURE__*/React.createElement("button", (0, _extends2.default)({