@atlaskit/color-picker 3.2.14 → 3.2.16

package/CHANGELOG.md

@@ -1,5 +1,22 @@
 # @atlaskit/color-picker
 
+## 3.2.16
+
+### Patch Changes
+
+- [#107395](https://stash.atlassian.com/projects/CONFCLOUD/repos/confluence-frontend/pull-requests/107395)
+  [`9a4ffd1c2458e`](https://stash.atlassian.com/projects/CONFCLOUD/repos/confluence-frontend/commits/9a4ffd1c2458e) -
+  added different label for radio item
+
+## 3.2.15
+
+### Patch Changes
+
+- [#107548](https://stash.atlassian.com/projects/CONFCLOUD/repos/confluence-frontend/pull-requests/107548)
+  [`86e1626ec32ff`](https://stash.atlassian.com/projects/CONFCLOUD/repos/confluence-frontend/commits/86e1626ec32ff) -
+  Clean up ff platform.color-picker-radio-button-functionality_6hkcy
+- Updated dependencies
+
 ## 3.2.14
 
 ### Patch Changes

package/codemods/1.0.0-popper-props.ts

@@ -1,71 +1,62 @@
 import {
-  addCommentToStartOfFile,
-  getJSXAttributesByName,
-  getDefaultSpecifierName,
-  hasImportDeclaration,
-  isUsingProp,
+	addCommentToStartOfFile,
+	getJSXAttributesByName,
+	getDefaultSpecifierName,
+	hasImportDeclaration,
+	isUsingProp,
 } from './utils/helpers';
 
 import {
-  type API,
-  type default as core,
-  type ASTPath,
-  type FileInfo,
-  type JSXElement,
-  type Options,
+	type API,
+	type default as core,
+	type ASTPath,
+	type FileInfo,
+	type JSXElement,
+	type Options,
 } from 'jscodeshift';
 import { type Collection } from 'jscodeshift/src/Collection';
 
-function updatePopperProps(
-  j: core.JSCodeshift,
-  source: Collection<any>,
-  specifier: string,
-) {
-  source.findJSXElements(specifier).forEach((path: ASTPath<JSXElement>) => {
-    if (
-      isUsingProp({
-        j,
-        base: source,
-        element: path,
-        propName: 'popperProps',
-      })
-    ) {
-      // Get value from prop
-      getJSXAttributesByName({
-        j,
-        element: path,
-        attributeName: 'popperProps',
-      }).forEach((attribute) => {
-        // convert positionFixed
-        j(attribute)
-          .find(j.JSXExpressionContainer)
-          .find(j.ObjectExpression)
-          .find(j.ObjectProperty)
-          // @ts-ignore
-          .filter((property) => property.node.key.name === 'positionFixed')
-          .forEach((property) => {
-            // @ts-ignore
-            const strategy = property.node.value.value ? 'fixed' : 'absolute';
-            property.replace(
-              j.objectProperty(
-                j.identifier('strategy'),
-                j.stringLiteral(strategy),
-              ),
-            );
-          });
+function updatePopperProps(j: core.JSCodeshift, source: Collection<any>, specifier: string) {
+	source.findJSXElements(specifier).forEach((path: ASTPath<JSXElement>) => {
+		if (
+			isUsingProp({
+				j,
+				base: source,
+				element: path,
+				propName: 'popperProps',
+			})
+		) {
+			// Get value from prop
+			getJSXAttributesByName({
+				j,
+				element: path,
+				attributeName: 'popperProps',
+			}).forEach((attribute) => {
+				// convert positionFixed
+				j(attribute)
+					.find(j.JSXExpressionContainer)
+					.find(j.ObjectExpression)
+					.find(j.ObjectProperty)
+					// @ts-ignore
+					.filter((property) => property.node.key.name === 'positionFixed')
+					.forEach((property) => {
+						// @ts-ignore
+						const strategy = property.node.value.value ? 'fixed' : 'absolute';
+						property.replace(j.objectProperty(j.identifier('strategy'), j.stringLiteral(strategy)));
+					});
 
-        // warn about modifiers prop
-        j(attribute)
-          .find(j.JSXExpressionContainer)
-          .find(j.ObjectExpression)
-          .find(j.ObjectProperty)
-          //@ts-ignore
-          .filter((property) => property.node.key.name === 'modifiers')
-          .forEach(() => {
-            addCommentToStartOfFile({
-              j,
-              base: source,
-              message: `
+				// warn about modifiers prop
+				j(attribute)
+					.find(j.JSXExpressionContainer)
+					.find(j.ObjectExpression)
+					.find(j.ObjectProperty)
+					//@ts-ignore
+					.filter((property) => property.node.key.name === 'modifiers')
+					.forEach(() => {
+						addCommentToStartOfFile({
+							j,
+							base: source,
+							message: `
                 Popper.js has been upgraded from 1.14.1 to 2.4.2,
                 and as a result the modifier prop has changed significantly. The format has been
                 changed from object of objects, to array of objects, with the key for each modifier
@@ -73,38 +64,34 @@ function updatePopperProps(
                 pair for configuration and other changes unique to each modifier.
                 Further details can be found in the popper docs: https://popper.js.org/docs/v2/modifiers/
                 `,
-            });
-          });
-      });
-    }
-  });
+						});
+					});
+			});
+		}
+	});
 }
 
-export default function transformer(
-  file: FileInfo,
-  { jscodeshift: j }: API,
-  options: Options,
-) {
-  const source = j(file.source);
+export default function transformer(file: FileInfo, { jscodeshift: j }: API, options: Options) {
+	const source = j(file.source);
 
-  // Exit early if not relevant
-  // We are doing this so we don't touch the formatting of unrelated files
-  if (!hasImportDeclaration(j, file.source, '@atlaskit/color-picker')) {
-    return file.source;
-  }
+	// Exit early if not relevant
+	// We are doing this so we don't touch the formatting of unrelated files
+	if (!hasImportDeclaration(j, file.source, '@atlaskit/color-picker')) {
+		return file.source;
+	}
 
-  // Get imported name for the component
-  const specifier = getDefaultSpecifierName({
-    j,
-    base: source,
-    packageName: '@atlaskit/color-picker',
-  });
-  if (!specifier) {
-    return file.source;
-  }
+	// Get imported name for the component
+	const specifier = getDefaultSpecifierName({
+		j,
+		base: source,
+		packageName: '@atlaskit/color-picker',
+	});
+	if (!specifier) {
+		return file.source;
+	}
 
-  // Convert boundaries prop
-  updatePopperProps(j, source, specifier);
+	// Convert boundaries prop
+	updatePopperProps(j, source, specifier);
 
-  return source.toSource(options.printOptions || { quote: 'single' });
+	return source.toSource(options.printOptions || { quote: 'single' });
 }

package/codemods/__tests__/1.0.0-popper-props.ts

@@ -5,43 +5,43 @@ import transformer from '../1.0.0-popper-props';
 const applyTransform = require('jscodeshift/dist/testUtils').applyTransform;
 
 type TestArgs = {
-  it: string;
-  original: string;
-  expected: string;
-  mode?: 'only' | 'skip';
-  before?: () => void;
-  after?: () => void;
+	it: string;
+	original: string;
+	expected: string;
+	mode?: 'only' | 'skip';
+	before?: () => void;
+	after?: () => void;
 };
 
 function noop() {}
 
 function check({
-  it: name,
-  original,
-  expected,
-  before = noop,
-  after = noop,
-  mode = undefined,
+	it: name,
+	original,
+	expected,
+	before = noop,
+	after = noop,
+	mode = undefined,
 }: TestArgs) {
-  const run = mode === 'only' ? it.only : mode === 'skip' ? it.skip : it;
-
-  run(name, () => {
-    before();
-    try {
-      const output: string = applyTransform(
-        { default: transformer, parser: 'tsx' },
-        {},
-        { source: original },
-      );
-      expect(output).toBe(expected.trim());
-    } catch (e) {
-      // a failed assertion will throw
-      after();
-      throw e;
-    }
-    // will only be hit if we don't throw
-    after();
-  });
+	const run = mode === 'only' ? it.only : mode === 'skip' ? it.skip : it;
+
+	run(name, () => {
+		before();
+		try {
+			const output: string = applyTransform(
+				{ default: transformer, parser: 'tsx' },
+				{},
+				{ source: original },
+			);
+			expect(output).toBe(expected.trim());
+		} catch (e) {
+			// a failed assertion will throw
+			after();
+			throw e;
+		}
+		// will only be hit if we don't throw
+		after();
+	});
 }
 
 /**
@@ -50,9 +50,9 @@ function check({
 */
 
 describe('Convert positionFixed:boolean to strategy:"fixed"|"absolute"', () => {
-  check({
-    it: 'Convert `positionFixed: true` to `strategy: "fixed"`',
-    original: `
+	check({
+		it: 'Convert `positionFixed: true` to `strategy: "fixed"`',
+		original: `
       import ColorPicker from '@atlaskit/color-picker';
 
       export default () => (
@@ -68,7 +68,7 @@ describe('Convert positionFixed:boolean to strategy:"fixed"|"absolute"', () => {
         />
       );
     `,
-    expected: `
+		expected: `
       import ColorPicker from '@atlaskit/color-picker';
 
       export default () => (
@@ -84,11 +84,11 @@ describe('Convert positionFixed:boolean to strategy:"fixed"|"absolute"', () => {
         />
       );
     `,
-  });
+	});
 
-  check({
-    it: 'Convert `positionFixed: {something truthy}` to `strategy: "fixed"`',
-    original: `
+	check({
+		it: 'Convert `positionFixed: {something truthy}` to `strategy: "fixed"`',
+		original: `
       import ColorPicker from '@atlaskit/color-picker';
 
       export default () => (
@@ -104,7 +104,7 @@ describe('Convert positionFixed:boolean to strategy:"fixed"|"absolute"', () => {
         />
       );
     `,
-    expected: `
+		expected: `
       import ColorPicker from '@atlaskit/color-picker';
 
       export default () => (
@@ -120,11 +120,11 @@ describe('Convert positionFixed:boolean to strategy:"fixed"|"absolute"', () => {
         />
       );
     `,
-  });
+	});
 
-  check({
-    it: 'Convert `positionFixed: false` to `strategy:"absolute"`',
-    original: `
+	check({
+		it: 'Convert `positionFixed: false` to `strategy:"absolute"`',
+		original: `
       import ColorPicker from '@atlaskit/color-picker';
 
       export default () => (
@@ -140,7 +140,7 @@ describe('Convert positionFixed:boolean to strategy:"fixed"|"absolute"', () => {
         />
       );
     `,
-    expected: `
+		expected: `
       import ColorPicker from '@atlaskit/color-picker';
 
       export default () => (
@@ -156,7 +156,7 @@ describe('Convert positionFixed:boolean to strategy:"fixed"|"absolute"', () => {
         />
       );
     `,
-  });
+	});
 });
 
 /**
@@ -165,9 +165,9 @@ describe('Convert positionFixed:boolean to strategy:"fixed"|"absolute"', () => {
 */
 
 describe('Warn use of the `modifiers` prop', () => {
-  check({
-    it: 'should add warning comment when using the modifier prop in `popperProps`',
-    original: `
+	check({
+		it: 'should add warning comment when using the modifier prop in `popperProps`',
+		original: `
       import ColorPicker from '@atlaskit/color-picker';
 
       export default () => (
@@ -183,7 +183,7 @@ describe('Warn use of the `modifiers` prop', () => {
         />
       );
     `,
-    expected: `
+		expected: `
     /* TODO: (from codemod) Popper.js has been upgraded from 1.14.1 to 2.4.2,
       and as a result the modifier prop has changed significantly. The format has been
       changed from object of objects, to array of objects, with the key for each modifier
@@ -205,5 +205,5 @@ describe('Warn use of the `modifiers` prop', () => {
         />
       );
     `,
-  });
+	});
 });