@atlaskit/editor-common 97.0.0 → 97.1.0

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # @atlaskit/editor-common
 
+## 97.1.0
+
+### Minor Changes
+
+- [`de09fa494ad65`](https://stash.atlassian.com/projects/CONFCLOUD/repos/confluence-frontend/commits/de09fa494ad65) -
+  FF cleanup for the use-effect-in-use-previous hook [GRAVITYAI-1925]
+
 ## 97.0.0
 
 ### Major Changes

package/dist/cjs/hooks/usePreviousState.js

@@ -5,7 +5,6 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.default = usePreviousState;
 var _react = require("react");
-var _platformFeatureFlags = require("@atlaskit/platform-feature-flags");
 /**
  *
  * Can be used to get the previous state of a prop.
@@ -18,14 +17,11 @@ var _platformFeatureFlags = require("@atlaskit/platform-feature-flags");
  */
 function usePreviousState(value, initialValue) {
   var ref = (0, _react.useRef)(initialValue);
-  if ((0, _platformFeatureFlags.fg)('use-effect-in-use-previous-props')) {
-    // eslint-disable-next-line react-hooks/rules-of-hooks
-    (0, _react.useEffect)(function () {
-      ref.current = value;
-    }, [value]);
-    return ref.current;
-  }
-  var prevValue = ref.current;
-  ref.current = value;
-  return prevValue;
+  (0, _react.useEffect)(function () {
+    // Setting the value to ref.current inside an effect, or otherwise the behaviour in staging/prod doesn't match with local
+    // In local, the double rendering in react dev mode causes the previous value to be the same as next value immediately after
+    // In prod and staging, double rendering is not done since that is a react prod build
+    ref.current = value;
+  }, [value]);
+  return ref.current;
 }

package/dist/cjs/monitoring/error.js

@@ -17,7 +17,7 @@ function _getRequireWildcardCache(e) { if ("function" != typeof WeakMap) return
 function _interopRequireWildcard(e, r) { if (!r && e && e.__esModule) return e; if (null === e || "object" != _typeof(e) && "function" != typeof e) return { default: e }; var t = _getRequireWildcardCache(r); if (t && t.has(e)) return t.get(e); var n = { __proto__: null }, a = Object.defineProperty && Object.getOwnPropertyDescriptor; for (var u in e) if ("default" !== u && {}.hasOwnProperty.call(e, u)) { var i = a ? Object.getOwnPropertyDescriptor(e, u) : null; i && (i.get || i.set) ? Object.defineProperty(n, u, i) : n[u] = e[u]; } return n.default = e, t && t.set(e, n), n; }
 var SENTRY_DSN = 'https://0b10c8e02fb44d8796c047b102c9bee8@o55978.ingest.sentry.io/4505129224110080';
 var packageName = 'editor-common'; // Sentry doesn't accept '/' in its releases https://docs.sentry.io/platforms/javascript/configuration/releases/
-var packageVersion = "97.0.0";
+var packageVersion = "97.1.0";
 var sanitiseSentryEvents = function sanitiseSentryEvents(data, _hint) {
   // Remove URL as it has UGC
   // TODO: Sanitise the URL instead of just removing it

package/dist/cjs/styles/index.js

@@ -243,6 +243,12 @@ Object.defineProperty(exports, "getOrderedListInlineStyles", {
     return _lists.getOrderedListInlineStyles;
   }
 });
+Object.defineProperty(exports, "getStatusSharedStyles", {
+  enumerable: true,
+  get: function get() {
+    return _status.getStatusSharedStyles;
+  }
+});
 Object.defineProperty(exports, "gridStyles", {
   enumerable: true,
   get: function get() {

package/dist/cjs/styles/shared/status.js

@@ -1,10 +1,26 @@
 "use strict";
 
+var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefault");
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
-exports.StatusSharedCssClassName = void 0;
+exports.getStatusSharedStyles = exports.StatusSharedCssClassName = void 0;
+var _defineProperty2 = _interopRequireDefault(require("@babel/runtime/helpers/defineProperty"));
+var _react = require("@emotion/react");
+var _platformFeatureFlags = require("@atlaskit/platform-feature-flags");
+// eslint-disable-next-line @atlaskit/ui-styling-standard/use-compiled -- Ignored via go/DSP-18766
+
 var StatusSharedCssClassName = exports.StatusSharedCssClassName = {
   STATUS_CONTAINER: 'statusView-content-wrap',
   STATUS_LOZENGE: 'status-lozenge-span'
+};
+var getStatusSharedStyles = exports.getStatusSharedStyles = function getStatusSharedStyles() {
+  return (
+    // eslint-disable-next-line @atlaskit/platform/ensure-feature-flag-prefix
+    (0, _platformFeatureFlags.fg)('platform-component-visual-refresh') ?
+    // eslint-disable-next-line @atlaskit/design-system/no-css-tagged-template-expression
+    (0, _react.css)((0, _defineProperty2.default)({}, ".".concat(StatusSharedCssClassName.STATUS_LOZENGE, " > span "), {
+      border: "1px solid ".concat("var(--ds-border-inverse, #FFF)")
+    })) : (0, _react.css)({})
+  );
 };

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

@@ -23,7 +23,7 @@ function _isNativeReflectConstruct() { try { var t = !Boolean.prototype.valueOf.
  * @jsx jsx
  */ // eslint-disable-next-line @atlaskit/ui-styling-standard/use-compiled -- Ignored via go/DSP-18766
 var packageName = "@atlaskit/editor-common";
-var packageVersion = "97.0.0";
+var packageVersion = "97.1.0";
 var halfFocusRing = 1;
 var dropOffset = '0, 8';
 var DropList = /*#__PURE__*/function (_Component) {

package/dist/es2019/hooks/usePreviousState.js

@@ -1,5 +1,4 @@
 import { useEffect, useRef } from 'react';
-import { fg } from '@atlaskit/platform-feature-flags';
 
 /**
  *
@@ -13,14 +12,11 @@ import { fg } from '@atlaskit/platform-feature-flags';
  */
 export default function usePreviousState(value, initialValue) {
   const ref = useRef(initialValue);
-  if (fg('use-effect-in-use-previous-props')) {
-    // eslint-disable-next-line react-hooks/rules-of-hooks
-    useEffect(() => {
-      ref.current = value;
-    }, [value]);
-    return ref.current;
-  }
-  const prevValue = ref.current;
-  ref.current = value;
-  return prevValue;
+  useEffect(() => {
+    // Setting the value to ref.current inside an effect, or otherwise the behaviour in staging/prod doesn't match with local
+    // In local, the double rendering in react dev mode causes the previous value to be the same as next value immediately after
+    // In prod and staging, double rendering is not done since that is a react prod build
+    ref.current = value;
+  }, [value]);
+  return ref.current;
 }

package/dist/es2019/monitoring/error.js

@@ -1,7 +1,7 @@
 import { isFedRamp } from './environment';
 const SENTRY_DSN = 'https://0b10c8e02fb44d8796c047b102c9bee8@o55978.ingest.sentry.io/4505129224110080';
 const packageName = 'editor-common'; // Sentry doesn't accept '/' in its releases https://docs.sentry.io/platforms/javascript/configuration/releases/
-const packageVersion = "97.0.0";
+const packageVersion = "97.1.0";
 const sanitiseSentryEvents = (data, _hint) => {
   // Remove URL as it has UGC
   // TODO: Sanitise the URL instead of just removing it

package/dist/es2019/styles/index.js

@@ -20,7 +20,7 @@ export { codeMarkSharedStyles } from './shared/code-mark';
 export { shadowSharedStyle } from './shared/shadow';
 export { dateSharedStyle, DateSharedCssClassName } from './shared/date';
 export { tasksAndDecisionsStyles, TaskDecisionSharedCssClassName } from './shared/task-decision';
-export { StatusSharedCssClassName } from './shared/status';
+export { StatusSharedCssClassName, getStatusSharedStyles } from './shared/status';
 export { smartCardSharedStyles, SmartCardSharedCssClassName } from './shared/smart-card';
 export { DropdownMenuSharedCssClassName } from './shared/dropdown-menu';
 export { CodeBlockSharedCssClassName, codeBlockSharedStyles, codeBlockInListSafariFix } from './shared/code-block';

package/dist/es2019/styles/shared/status.js

@@ -1,4 +1,17 @@
+// eslint-disable-next-line @atlaskit/ui-styling-standard/use-compiled -- Ignored via go/DSP-18766
+import { css } from '@emotion/react';
+import { fg } from '@atlaskit/platform-feature-flags';
 export const StatusSharedCssClassName = {
   STATUS_CONTAINER: 'statusView-content-wrap',
   STATUS_LOZENGE: 'status-lozenge-span'
-};
+};
+export const getStatusSharedStyles = () =>
+// eslint-disable-next-line @atlaskit/platform/ensure-feature-flag-prefix
+fg('platform-component-visual-refresh') ?
+// eslint-disable-next-line @atlaskit/design-system/no-css-tagged-template-expression
+css({
+  // eslint-disable-next-line @atlaskit/ui-styling-standard/no-nested-selectors, @atlaskit/ui-styling-standard/no-unsafe-values
+  [`.${StatusSharedCssClassName.STATUS_LOZENGE} > span `]: {
+    border: `1px solid ${"var(--ds-border-inverse, #FFF)"}`
+  }
+}) : css({});

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

@@ -13,7 +13,7 @@ import withAnalyticsContext from '@atlaskit/analytics-next/withAnalyticsContext'
 import withAnalyticsEvents from '@atlaskit/analytics-next/withAnalyticsEvents';
 import Layer from '../Layer';
 const packageName = "@atlaskit/editor-common";
-const packageVersion = "97.0.0";
+const packageVersion = "97.1.0";
 const halfFocusRing = 1;
 const dropOffset = '0, 8';
 class DropList extends Component {

package/dist/esm/hooks/usePreviousState.js

@@ -1,5 +1,4 @@
 import { useEffect, useRef } from 'react';
-import { fg } from '@atlaskit/platform-feature-flags';
 
 /**
  *
@@ -13,14 +12,11 @@ import { fg } from '@atlaskit/platform-feature-flags';
  */
 export default function usePreviousState(value, initialValue) {
   var ref = useRef(initialValue);
-  if (fg('use-effect-in-use-previous-props')) {
-    // eslint-disable-next-line react-hooks/rules-of-hooks
-    useEffect(function () {
-      ref.current = value;
-    }, [value]);
-    return ref.current;
-  }
-  var prevValue = ref.current;
-  ref.current = value;
-  return prevValue;
+  useEffect(function () {
+    // Setting the value to ref.current inside an effect, or otherwise the behaviour in staging/prod doesn't match with local
+    // In local, the double rendering in react dev mode causes the previous value to be the same as next value immediately after
+    // In prod and staging, double rendering is not done since that is a react prod build
+    ref.current = value;
+  }, [value]);
+  return ref.current;
 }

package/dist/esm/monitoring/error.js

@@ -7,7 +7,7 @@ function _objectSpread(e) { for (var r = 1; r < arguments.length; r++) { var t =
 import { isFedRamp } from './environment';
 var SENTRY_DSN = 'https://0b10c8e02fb44d8796c047b102c9bee8@o55978.ingest.sentry.io/4505129224110080';
 var packageName = 'editor-common'; // Sentry doesn't accept '/' in its releases https://docs.sentry.io/platforms/javascript/configuration/releases/
-var packageVersion = "97.0.0";
+var packageVersion = "97.1.0";
 var sanitiseSentryEvents = function sanitiseSentryEvents(data, _hint) {
   // Remove URL as it has UGC
   // TODO: Sanitise the URL instead of just removing it

package/dist/esm/styles/index.js

@@ -20,7 +20,7 @@ export { codeMarkSharedStyles } from './shared/code-mark';
 export { shadowSharedStyle } from './shared/shadow';
 export { dateSharedStyle, DateSharedCssClassName } from './shared/date';
 export { tasksAndDecisionsStyles, TaskDecisionSharedCssClassName } from './shared/task-decision';
-export { StatusSharedCssClassName } from './shared/status';
+export { StatusSharedCssClassName, getStatusSharedStyles } from './shared/status';
 export { smartCardSharedStyles, SmartCardSharedCssClassName } from './shared/smart-card';
 export { DropdownMenuSharedCssClassName } from './shared/dropdown-menu';
 export { CodeBlockSharedCssClassName, codeBlockSharedStyles, codeBlockInListSafariFix } from './shared/code-block';

package/dist/esm/styles/shared/status.js

@@ -1,4 +1,18 @@
+import _defineProperty from "@babel/runtime/helpers/defineProperty";
+// eslint-disable-next-line @atlaskit/ui-styling-standard/use-compiled -- Ignored via go/DSP-18766
+import { css } from '@emotion/react';
+import { fg } from '@atlaskit/platform-feature-flags';
 export var StatusSharedCssClassName = {
   STATUS_CONTAINER: 'statusView-content-wrap',
   STATUS_LOZENGE: 'status-lozenge-span'
+};
+export var getStatusSharedStyles = function getStatusSharedStyles() {
+  return (
+    // eslint-disable-next-line @atlaskit/platform/ensure-feature-flag-prefix
+    fg('platform-component-visual-refresh') ?
+    // eslint-disable-next-line @atlaskit/design-system/no-css-tagged-template-expression
+    css(_defineProperty({}, ".".concat(StatusSharedCssClassName.STATUS_LOZENGE, " > span "), {
+      border: "1px solid ".concat("var(--ds-border-inverse, #FFF)")
+    })) : css({})
+  );
 };

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

@@ -20,7 +20,7 @@ import withAnalyticsContext from '@atlaskit/analytics-next/withAnalyticsContext'
 import withAnalyticsEvents from '@atlaskit/analytics-next/withAnalyticsEvents';
 import Layer from '../Layer';
 var packageName = "@atlaskit/editor-common";
-var packageVersion = "97.0.0";
+var packageVersion = "97.1.0";
 var halfFocusRing = 1;
 var dropOffset = '0, 8';
 var DropList = /*#__PURE__*/function (_Component) {

package/dist/types/styles/index.d.ts

@@ -17,7 +17,7 @@ export { codeMarkSharedStyles } from './shared/code-mark';
 export { shadowSharedStyle } from './shared/shadow';
 export { dateSharedStyle, DateSharedCssClassName } from './shared/date';
 export { tasksAndDecisionsStyles, TaskDecisionSharedCssClassName } from './shared/task-decision';
-export { StatusSharedCssClassName } from './shared/status';
+export { StatusSharedCssClassName, getStatusSharedStyles } from './shared/status';
 export { smartCardSharedStyles, SmartCardSharedCssClassName } from './shared/smart-card';
 export { DropdownMenuSharedCssClassName } from './shared/dropdown-menu';
 export { CodeBlockSharedCssClassName, codeBlockSharedStyles, codeBlockInListSafariFix, } from './shared/code-block';

package/dist/types/styles/shared/status.d.ts

@@ -2,3 +2,4 @@ export declare const StatusSharedCssClassName: {
     STATUS_CONTAINER: string;
     STATUS_LOZENGE: string;
 };
+export declare const getStatusSharedStyles: () => import("@emotion/react").SerializedStyles;

package/dist/types-ts4.5/styles/index.d.ts

@@ -17,7 +17,7 @@ export { codeMarkSharedStyles } from './shared/code-mark';
 export { shadowSharedStyle } from './shared/shadow';
 export { dateSharedStyle, DateSharedCssClassName } from './shared/date';
 export { tasksAndDecisionsStyles, TaskDecisionSharedCssClassName } from './shared/task-decision';
-export { StatusSharedCssClassName } from './shared/status';
+export { StatusSharedCssClassName, getStatusSharedStyles } from './shared/status';
 export { smartCardSharedStyles, SmartCardSharedCssClassName } from './shared/smart-card';
 export { DropdownMenuSharedCssClassName } from './shared/dropdown-menu';
 export { CodeBlockSharedCssClassName, codeBlockSharedStyles, codeBlockInListSafariFix, } from './shared/code-block';

package/dist/types-ts4.5/styles/shared/status.d.ts

@@ -2,3 +2,4 @@ export declare const StatusSharedCssClassName: {
     STATUS_CONTAINER: string;
     STATUS_LOZENGE: string;
 };
+export declare const getStatusSharedStyles: () => import("@emotion/react").SerializedStyles;

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@atlaskit/editor-common",
-	"version": "97.0.0",
+	"version": "97.1.0",
 	"description": "A package that contains common classes and components for editor and renderer",
 	"publishConfig": {
 		"registry": "https://registry.npmjs.org/"
@@ -257,9 +257,6 @@
 		"platform_editor_react18_phase2_v2": {
 			"type": "boolean"
 		},
-		"use-effect-in-use-previous-props": {
-			"type": "boolean"
-		},
 		"platform_editor_react18_extension_component_v2": {
 			"type": "boolean"
 		},
@@ -322,6 +319,9 @@
 		},
 		"platform_editor_support_media_badge_visibility": {
 			"type": "boolean"
+		},
+		"platform-component-visual-refresh": {
+			"type": "boolean"
 		}
 	}
 }