@atlaskit/renderer 128.10.6 → 128.11.1

package/CHANGELOG.md

@@ -1,5 +1,25 @@
 # @atlaskit/renderer
 
+## 128.11.1
+
+### Patch Changes
+
+- [`c4c9b75b91fe3`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/c4c9b75b91fe3) -
+  Removed feature flag `platform_editor_fix_wide_media_in_renderer`. The fix for wide media in
+  renderer (skipping width override for pixel-based resizing) is now the default behavior.
+- Updated dependencies
+
+## 128.11.0
+
+### Minor Changes
+
+- [`65f1d80415ae8`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/65f1d80415ae8) -
+  EDITOR-679 improve reliability of media provider
+
+### Patch Changes
+
+- Updated dependencies
+
 ## 128.10.6
 
 ### Patch Changes

package/dist/cjs/react/utils/EditorMediaClientProvider.js

@@ -10,6 +10,7 @@ var _slicedToArray2 = _interopRequireDefault(require("@babel/runtime/helpers/sli
 var _react = _interopRequireWildcard(require("react"));
 var _mediaClientReact = require("@atlaskit/media-client-react");
 var _providerFactory = require("@atlaskit/editor-common/provider-factory");
+var _expValEquals = require("@atlaskit/tmp-editor-statsig/exp-val-equals");
 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 EditorMediaClientProvider = exports.EditorMediaClientProvider = function EditorMediaClientProvider(_ref) {
   var children = _ref.children,
@@ -18,13 +19,24 @@ var EditorMediaClientProvider = exports.EditorMediaClientProvider = function Edi
     _useState2 = (0, _slicedToArray2.default)(_useState, 2),
     mediaClientConfig = _useState2[0],
     setMediaClientConfig = _useState2[1];
+  var providerFactory = (0, _providerFactory.useProviderFactory)();
   var mediaProvider = (0, _providerFactory.useProviderLayout)('mediaProvider');
 
   /**
-   * If a mediaClientConfig is provided then we will force
-   * skip the mediaClient from context
+   * Whether this renderer has its own media provider and should never inherit
+   * the mediaClient from a parent renderer's context.
+   *
+   * We use providerFactory.hasProvider() rather and checking the mediaProvider
+   * state value, because useProviderLayout subscribes via useLayoutEffect —
+   * meaning mediaProvider state is always undefined on the first render, even
+   * if the ProviderFactory already has a provider registered. This would cause
+   * shouldSkipContext to be false on the first render, incorrectly allowing the
+   * inner renderer to inherit the outer renderer's MediaClientContext (which
+   * carries the wrong media token for this page).
+   *
+   * hasProvider() is synchronous and correct from render 1, closing that window.
    */
-  var shouldSkipContext = Boolean((ssr === null || ssr === void 0 ? void 0 : ssr.config) || mediaProvider);
+  var shouldSkipContext = (0, _expValEquals.expValEquals)('platform_editor_media_reliability_enhancements', 'isEnabled', true) ? Boolean((ssr === null || ssr === void 0 ? void 0 : ssr.config) || providerFactory.hasProvider('mediaProvider') || mediaProvider) : Boolean((ssr === null || ssr === void 0 ? void 0 : ssr.config) || mediaProvider);
   var contextMediaClient = (0, _react.useContext)(_mediaClientReact.MediaClientContext);
 
   // MediaClientProvider currently requires a mediaClientConfig

package/dist/cjs/ui/Renderer/breakout-ssr.js

@@ -51,7 +51,6 @@ function BreakoutSSRInlineScript(_ref) {
 function createBreakoutInlineScript(id, shouldSkipScript) {
   var flags = {
     platform_editor_fix_media_in_renderer: (0, _platformFeatureFlags.fg)('platform_editor_fix_media_in_renderer'),
-    platform_editor_fix_wide_media_in_renderer: (0, _platformFeatureFlags.fg)('platform_editor_fix_wide_media_in_renderer'),
     platform_editor_renderer_extension_width_fix: (0, _expValEquals.expValEquals)('platform_editor_renderer_extension_width_fix', 'isEnabled', true)
   };
   return "(function(window){\nif(typeof window !== 'undefined' && window.__RENDERER_BYPASS_BREAKOUT_SSR__) { return; }\n".concat(breakoutInlineScriptContext, ";\n(").concat(applyBreakoutAfterSSR.toString(), ")(\"").concat(id, "\", breakoutConsts, ").concat(JSON.stringify(shouldSkipScript), ", ").concat(JSON.stringify(flags), ");\n})(window);\n");
@@ -195,12 +194,12 @@ function applyBreakoutAfterSSR(id, breakoutConsts, shouldSkipBreakoutScript, fla
 
     // Pixel based resizing has width set in pixels based on its width attribute
     // Thus, no need to override width
-    if (flags['platform_editor_fix_wide_media_in_renderer'] && isPixelBasedResizing) {
+    if (isPixelBasedResizing) {
       return;
     }
     if (WIDE_LAYOUT_MODES.includes(mode)) {
       card.style.width = '100%';
-    } else if (width && (!isPixelBasedResizing || flags['platform_editor_fix_wide_media_in_renderer'])) {
+    } else if (width) {
       card.style.width = "".concat(width, "%");
     }
   };

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

@@ -71,7 +71,7 @@ var DEGRADED_SEVERITY_THRESHOLD = exports.DEGRADED_SEVERITY_THRESHOLD = 3000;
 var TABLE_INFO_TIMEOUT = 10000;
 var RENDER_EVENT_SAMPLE_RATE = 0.2;
 var packageName = "@atlaskit/renderer";
-var packageVersion = "128.10.5";
+var packageVersion = "128.11.0";
 var setAsQueryContainerStyles = (0, _react2.css)({
   containerName: 'ak-renderer-wrapper',
   containerType: 'inline-size'

package/dist/es2019/react/utils/EditorMediaClientProvider.js

@@ -1,18 +1,30 @@
 import React, { useContext, useLayoutEffect, useMemo, useState } from 'react';
 import { MediaClientContext, getMediaClient } from '@atlaskit/media-client-react';
-import { useProviderLayout } from '@atlaskit/editor-common/provider-factory';
+import { useProviderFactory, useProviderLayout } from '@atlaskit/editor-common/provider-factory';
+import { expValEquals } from '@atlaskit/tmp-editor-statsig/exp-val-equals';
 export const EditorMediaClientProvider = ({
   children,
   ssr
 }) => {
   const [mediaClientConfig, setMediaClientConfig] = useState();
+  const providerFactory = useProviderFactory();
   const mediaProvider = useProviderLayout('mediaProvider');
 
   /**
-   * If a mediaClientConfig is provided then we will force
-   * skip the mediaClient from context
+   * Whether this renderer has its own media provider and should never inherit
+   * the mediaClient from a parent renderer's context.
+   *
+   * We use providerFactory.hasProvider() rather and checking the mediaProvider
+   * state value, because useProviderLayout subscribes via useLayoutEffect —
+   * meaning mediaProvider state is always undefined on the first render, even
+   * if the ProviderFactory already has a provider registered. This would cause
+   * shouldSkipContext to be false on the first render, incorrectly allowing the
+   * inner renderer to inherit the outer renderer's MediaClientContext (which
+   * carries the wrong media token for this page).
+   *
+   * hasProvider() is synchronous and correct from render 1, closing that window.
    */
-  const shouldSkipContext = Boolean((ssr === null || ssr === void 0 ? void 0 : ssr.config) || mediaProvider);
+  const shouldSkipContext = expValEquals('platform_editor_media_reliability_enhancements', 'isEnabled', true) ? Boolean((ssr === null || ssr === void 0 ? void 0 : ssr.config) || providerFactory.hasProvider('mediaProvider') || mediaProvider) : Boolean((ssr === null || ssr === void 0 ? void 0 : ssr.config) || mediaProvider);
   const contextMediaClient = useContext(MediaClientContext);
 
   // MediaClientProvider currently requires a mediaClientConfig

package/dist/es2019/ui/Renderer/breakout-ssr.js

@@ -43,7 +43,6 @@ export function BreakoutSSRInlineScript({
 export function createBreakoutInlineScript(id, shouldSkipScript) {
   const flags = {
     platform_editor_fix_media_in_renderer: fg('platform_editor_fix_media_in_renderer'),
-    platform_editor_fix_wide_media_in_renderer: fg('platform_editor_fix_wide_media_in_renderer'),
     platform_editor_renderer_extension_width_fix: expValEquals('platform_editor_renderer_extension_width_fix', 'isEnabled', true)
   };
   return `(function(window){
@@ -198,12 +197,12 @@ function applyBreakoutAfterSSR(id, breakoutConsts, shouldSkipBreakoutScript, fla
 
     // Pixel based resizing has width set in pixels based on its width attribute
     // Thus, no need to override width
-    if (flags['platform_editor_fix_wide_media_in_renderer'] && isPixelBasedResizing) {
+    if (isPixelBasedResizing) {
       return;
     }
     if (WIDE_LAYOUT_MODES.includes(mode)) {
       card.style.width = '100%';
-    } else if (width && (!isPixelBasedResizing || flags['platform_editor_fix_wide_media_in_renderer'])) {
+    } else if (width) {
       card.style.width = `${width}%`;
     }
   };

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

@@ -57,7 +57,7 @@ export const DEGRADED_SEVERITY_THRESHOLD = 3000;
 const TABLE_INFO_TIMEOUT = 10000;
 const RENDER_EVENT_SAMPLE_RATE = 0.2;
 const packageName = "@atlaskit/renderer";
-const packageVersion = "128.10.5";
+const packageVersion = "128.11.0";
 const setAsQueryContainerStyles = css({
   containerName: 'ak-renderer-wrapper',
   containerType: 'inline-size'

package/dist/esm/react/utils/EditorMediaClientProvider.js

@@ -1,7 +1,8 @@
 import _slicedToArray from "@babel/runtime/helpers/slicedToArray";
 import React, { useContext, useLayoutEffect, useMemo, useState } from 'react';
 import { MediaClientContext, getMediaClient } from '@atlaskit/media-client-react';
-import { useProviderLayout } from '@atlaskit/editor-common/provider-factory';
+import { useProviderFactory, useProviderLayout } from '@atlaskit/editor-common/provider-factory';
+import { expValEquals } from '@atlaskit/tmp-editor-statsig/exp-val-equals';
 export var EditorMediaClientProvider = function EditorMediaClientProvider(_ref) {
   var children = _ref.children,
     ssr = _ref.ssr;
@@ -9,13 +10,24 @@ export var EditorMediaClientProvider = function EditorMediaClientProvider(_ref)
     _useState2 = _slicedToArray(_useState, 2),
     mediaClientConfig = _useState2[0],
     setMediaClientConfig = _useState2[1];
+  var providerFactory = useProviderFactory();
   var mediaProvider = useProviderLayout('mediaProvider');
 
   /**
-   * If a mediaClientConfig is provided then we will force
-   * skip the mediaClient from context
+   * Whether this renderer has its own media provider and should never inherit
+   * the mediaClient from a parent renderer's context.
+   *
+   * We use providerFactory.hasProvider() rather and checking the mediaProvider
+   * state value, because useProviderLayout subscribes via useLayoutEffect —
+   * meaning mediaProvider state is always undefined on the first render, even
+   * if the ProviderFactory already has a provider registered. This would cause
+   * shouldSkipContext to be false on the first render, incorrectly allowing the
+   * inner renderer to inherit the outer renderer's MediaClientContext (which
+   * carries the wrong media token for this page).
+   *
+   * hasProvider() is synchronous and correct from render 1, closing that window.
    */
-  var shouldSkipContext = Boolean((ssr === null || ssr === void 0 ? void 0 : ssr.config) || mediaProvider);
+  var shouldSkipContext = expValEquals('platform_editor_media_reliability_enhancements', 'isEnabled', true) ? Boolean((ssr === null || ssr === void 0 ? void 0 : ssr.config) || providerFactory.hasProvider('mediaProvider') || mediaProvider) : Boolean((ssr === null || ssr === void 0 ? void 0 : ssr.config) || mediaProvider);
   var contextMediaClient = useContext(MediaClientContext);
 
   // MediaClientProvider currently requires a mediaClientConfig

package/dist/esm/ui/Renderer/breakout-ssr.js

@@ -42,7 +42,6 @@ export function BreakoutSSRInlineScript(_ref) {
 export function createBreakoutInlineScript(id, shouldSkipScript) {
   var flags = {
     platform_editor_fix_media_in_renderer: fg('platform_editor_fix_media_in_renderer'),
-    platform_editor_fix_wide_media_in_renderer: fg('platform_editor_fix_wide_media_in_renderer'),
     platform_editor_renderer_extension_width_fix: expValEquals('platform_editor_renderer_extension_width_fix', 'isEnabled', true)
   };
   return "(function(window){\nif(typeof window !== 'undefined' && window.__RENDERER_BYPASS_BREAKOUT_SSR__) { return; }\n".concat(breakoutInlineScriptContext, ";\n(").concat(applyBreakoutAfterSSR.toString(), ")(\"").concat(id, "\", breakoutConsts, ").concat(JSON.stringify(shouldSkipScript), ", ").concat(JSON.stringify(flags), ");\n})(window);\n");
@@ -186,12 +185,12 @@ function applyBreakoutAfterSSR(id, breakoutConsts, shouldSkipBreakoutScript, fla
 
     // Pixel based resizing has width set in pixels based on its width attribute
     // Thus, no need to override width
-    if (flags['platform_editor_fix_wide_media_in_renderer'] && isPixelBasedResizing) {
+    if (isPixelBasedResizing) {
       return;
     }
     if (WIDE_LAYOUT_MODES.includes(mode)) {
       card.style.width = '100%';
-    } else if (width && (!isPixelBasedResizing || flags['platform_editor_fix_wide_media_in_renderer'])) {
+    } else if (width) {
       card.style.width = "".concat(width, "%");
     }
   };

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

@@ -62,7 +62,7 @@ export var DEGRADED_SEVERITY_THRESHOLD = 3000;
 var TABLE_INFO_TIMEOUT = 10000;
 var RENDER_EVENT_SAMPLE_RATE = 0.2;
 var packageName = "@atlaskit/renderer";
-var packageVersion = "128.10.5";
+var packageVersion = "128.11.0";
 var setAsQueryContainerStyles = css({
   containerName: 'ak-renderer-wrapper',
   containerType: 'inline-size'

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@atlaskit/renderer",
-	"version": "128.10.6",
+	"version": "128.11.1",
 	"description": "Renderer component",
 	"publishConfig": {
 		"registry": "https://registry.npmjs.org/"
@@ -37,7 +37,7 @@
 		"@atlaskit/editor-prosemirror": "^7.3.0",
 		"@atlaskit/editor-shared-styles": "^3.10.0",
 		"@atlaskit/editor-smart-link-draggable": "^0.5.0",
-		"@atlaskit/emoji": "^69.12.0",
+		"@atlaskit/emoji": "^70.0.0",
 		"@atlaskit/feature-gate-js-client": "^5.5.0",
 		"@atlaskit/icon": "^34.1.0",
 		"@atlaskit/link": "^3.4.0",
@@ -56,10 +56,10 @@
 		"@atlaskit/pragmatic-drag-and-drop": "^1.7.0",
 		"@atlaskit/react-ufo": "^5.13.0",
 		"@atlaskit/smart-card": "^44.0.0",
-		"@atlaskit/status": "^3.2.0",
-		"@atlaskit/task-decision": "^19.3.0",
+		"@atlaskit/status": "^4.0.0",
+		"@atlaskit/task-decision": "^20.0.0",
 		"@atlaskit/theme": "^23.0.0",
-		"@atlaskit/tmp-editor-statsig": "^62.1.0",
+		"@atlaskit/tmp-editor-statsig": "^62.2.0",
 		"@atlaskit/tokens": "^13.0.0",
 		"@atlaskit/tooltip": "^21.1.0",
 		"@atlaskit/visually-hidden": "^3.0.0",
@@ -89,7 +89,7 @@
 		"@atlaskit/media-core": "^37.0.0",
 		"@atlaskit/media-integration-test-helpers": "workspace:^",
 		"@atlaskit/media-test-helpers": "^40.0.0",
-		"@atlaskit/mention": "^24.6.0",
+		"@atlaskit/mention": "^25.0.0",
 		"@atlaskit/modal-dialog": "^14.15.0",
 		"@atlaskit/navigation-system": "^7.3.0",
 		"@atlaskit/profilecard": "^24.49.0",
@@ -160,9 +160,6 @@
 		"platform_editor_fix_media_in_renderer": {
 			"type": "boolean"
 		},
-		"platform_editor_fix_wide_media_in_renderer": {
-			"type": "boolean"
-		},
 		"platform_editor_content_mode_button_mvp": {
 			"type": "boolean"
 		},