fix: "RequiredProps" unions in TextField & TextArea

salute-developers · Nov 26, 2024 · abb4a10 · abb4a10
1 parent 96fa7d6
commit abb4a10
Show file tree

Hide file tree

Showing 24 changed files with 5,296 additions and 15,563 deletions.
diff --git a/...TextArea.component-test.tsx/plasma-b2c TextArea -- _required -- _sizel.snap.png b/...TextArea.component-test.tsx/plasma-b2c TextArea -- _required -- _sizel.snap.png
diff --git a/...TextArea.component-test.tsx/plasma-b2c TextArea -- _required -- _sizem.snap.png b/...TextArea.component-test.tsx/plasma-b2c TextArea -- _required -- _sizem.snap.png
diff --git a/...TextArea.component-test.tsx/plasma-b2c TextArea -- _required -- _sizes.snap.png b/...TextArea.component-test.tsx/plasma-b2c TextArea -- _required -- _sizes.snap.png
diff --git a/...extArea.component-test.tsx/plasma-b2c TextArea -- _required -- _sizexs.snap.png b/...extArea.component-test.tsx/plasma-b2c TextArea -- _required -- _sizexs.snap.png
diff --git a/...TextArea.component-test.tsx/plasma-web TextArea -- _required -- _sizel.snap.png b/...TextArea.component-test.tsx/plasma-web TextArea -- _required -- _sizel.snap.png
diff --git a/...TextArea.component-test.tsx/plasma-web TextArea -- _required -- _sizem.snap.png b/...TextArea.component-test.tsx/plasma-web TextArea -- _required -- _sizem.snap.png
diff --git a/...TextArea.component-test.tsx/plasma-web TextArea -- _required -- _sizes.snap.png b/...TextArea.component-test.tsx/plasma-web TextArea -- _required -- _sizes.snap.png
diff --git a/...extArea.component-test.tsx/plasma-web TextArea -- _required -- _sizexs.snap.png b/...extArea.component-test.tsx/plasma-web TextArea -- _required -- _sizexs.snap.png
diff --git a/packages/plasma-b2c/api/plasma-b2c.api.md b/packages/plasma-b2c/api/plasma-b2c.api.md
diff --git a/packages/plasma-b2c/src/components/TextArea/TextArea.component-test.tsx b/packages/plasma-b2c/src/components/TextArea/TextArea.component-test.tsx
@@ -240,9 +240,9 @@ describe('plasma-b2c: TextArea', () => {
 
         const cases = [
             { required: true, labelPlacement: 'outer' },
-            { required: true, requiredPlacement: 'right', labelPlacement: 'outer' },
+            { required: true, requiredPlacement: 'left', labelPlacement: 'outer' },
             { required: true, labelPlacement: 'inner' },
-            { required: true, requiredPlacement: 'right', labelPlacement: 'inner' },
+            { required: true, requiredPlacement: 'left', labelPlacement: 'inner' },
         ];
 
         sizes.forEach((size) => {
@@ -334,9 +334,9 @@ describe('plasma-b2c: TextArea', () => {
 
         const cases = [
             { required: true, labelPlacement: 'outer' },
-            { required: true, requiredPlacement: 'right', labelPlacement: 'outer' },
+            { required: true, requiredPlacement: 'left', labelPlacement: 'outer' },
             { required: true, labelPlacement: 'inner' },
-            { required: true, requiredPlacement: 'right', labelPlacement: 'inner' },
+            { required: true, requiredPlacement: 'left', labelPlacement: 'inner' },
         ];
 
         sizes.forEach((size) => {

diff --git a/packages/plasma-b2c/src/components/TextArea/index.ts b/packages/plasma-b2c/src/components/TextArea/index.ts
@@ -2,39 +2,11 @@ import type { TextAreaProps as TextAreaPropsBase } from '@salutejs/plasma-hope';
 import type { PopoverPlacement, PopoverPlacementBasic } from '@salutejs/plasma-new-hope';
 import type { ReactNode } from 'react';
 
-export { TextArea } from './TextArea';
+import { TextArea } from './TextArea';
 
-type RequiredProps =
-    | {
-          /**
-           * Флаг обязательности поля
-           */
-          required: true;
-          /**
-           * Задает выравнивание индикатора обязательности поля
-           * @default right
-           */
-          requiredPlacement?: 'left' | 'right';
-          /**
-           * Флаг необязательности поля
-           */
-          optional?: false;
-      }
-    | {
-          /**
-           * Флаг необязательности поля
-           */
-          optional?: true;
-          /**
-           * Флаг обязательности поля
-           */
-          required?: false;
-          /**
-           * Задает выравнивание индикатора обязательности поля
-           * @default right
-           */
-          requiredPlacement?: never;
-      };
+export { TextArea };
+
+type newHopeTextAreaProps = React.ComponentProps<typeof TextArea>;
 
 type HintProps =
     | {
@@ -122,16 +94,7 @@ type ClearProps =
           hasDivider?: never;
       };
 
-export type TextAreaProps = TextAreaPropsBase & {
-    /**
-     * @deprecated не используется в компоненте
-     */
-    resize?: 'none' | 'both' | 'horizontal' | 'vertical';
-    /**
-     * Метка-подпись к элементу справа.
-     */
-    titleCaption?: ReactNode;
-} & RequiredProps &
+export type TextAreaProps = TextAreaPropsBase &
+    Pick<newHopeTextAreaProps, 'title' | 'titleCaption' | 'required' | 'requiredPlacement' | 'optional'> &
     ClearProps &
-    RequiredProps &
     HintProps;
diff --git a/packages/plasma-b2c/src/components/TextField/TextField.tsx b/packages/plasma-b2c/src/components/TextField/TextField.tsx
@@ -10,38 +10,6 @@ export const TextFieldComponent = component(mergedConfig);
 
 type newHopeTextFieldProps = ComponentProps<typeof TextFieldComponent>;
 
-export type RequiredProps =
-    | {
-          /**
-           * Флаг обязательности поля
-           */
-          required: true;
-          /**
-           * Задает выравнивание индикатора обязательности поля
-           * @default right
-           */
-          requiredPlacement?: 'left' | 'right';
-          /**
-           * Флаг необязательности поля
-           */
-          optional?: false;
-      }
-    | {
-          /**
-           * Флаг обязательности поля
-           */
-          required?: false;
-          /**
-           * Задает выравнивание индикатора обязательности поля
-           * @default right
-           */
-          requiredPlacement?: never;
-          /**
-           * Флаг необязательности поля
-           */
-          optional?: boolean;
-      };
-
 type ClearProps =
     | {
           /**
@@ -133,8 +101,10 @@ export type CustomTextFieldProps = (TextFieldProps &
         | 'titleCaption'
         | 'labelPlacement'
         | 'keepPlaceholder'
+        | 'required'
+        | 'requiredPlacement'
+        | 'optional'
     >) &
-    RequiredProps &
     ClearProps &
     HintProps;
 

diff --git a/packages/plasma-core/src/components/TextArea/TextArea.component-test.tsx b/packages/plasma-core/src/components/TextArea/TextArea.component-test.tsx
@@ -205,9 +205,9 @@ describe('plasma-core: TextArea', () => {
 
         const cases = [
             { required: true, labelPlacement: 'outer' },
-            { required: true, requiredPlacement: 'right', labelPlacement: 'outer' },
+            { required: true, requiredPlacement: 'left', labelPlacement: 'outer' },
             { required: true, labelPlacement: 'inner' },
-            { required: true, requiredPlacement: 'right', labelPlacement: 'inner' },
+            { required: true, requiredPlacement: 'left', labelPlacement: 'inner' },
         ];
 
         sizes.forEach((size) => {

diff --git a/packages/plasma-new-hope/src/components/TextArea/TextArea.types.tsx b/packages/plasma-new-hope/src/components/TextArea/TextArea.types.tsx
@@ -77,37 +77,21 @@ export type TextAreaDimensionsProps = OneOf<
     ClearProps
 >;
 
-type RequiredProps =
-    | {
-          /**
-           * Флаг обязательности поля
-           */
-          required: true;
-          /**
-           * Задает выравнивание индикатора обязательности поля
-           * @default right
-           */
-          requiredPlacement?: 'left' | 'right';
-          /**
-           * Флаг необязательности поля
-           */
-          optional?: false;
-      }
-    | {
-          /**
-           * Флаг необязательности поля
-           */
-          optional?: true;
-          /**
-           * Флаг обязательности поля
-           */
-          required?: false;
-          /**
-           * Задает выравнивание индикатора обязательности поля
-           * @default right
-           */
-          requiredPlacement?: never;
-      };
+type RequiredProps = {
+    /**
+     * Флаг обязательности поля
+     */
+    required?: boolean;
+    /**
+     * Задает выравнивание индикатора обязательности поля
+     * @default right
+     */
+    requiredPlacement?: 'left' | 'right';
+    /**
+     * Флаг необязательности поля
+     */
+    optional?: boolean;
+};
 
 type HintProps =
     | {

diff --git a/packages/plasma-new-hope/src/components/TextField/TextField.types.ts b/packages/plasma-new-hope/src/components/TextField/TextField.types.ts
@@ -60,37 +60,21 @@ type TextFieldChipProps =
           chipType?: 'default' | 'text';
       };
 
-export type RequiredProps =
-    | {
-          /**
-           * Флаг обязательности поля
-           */
-          required: true;
-          /**
-           * Задает выравнивание индикатора обязательности поля
-           * @default right
-           */
-          requiredPlacement?: 'left' | 'right';
-          /**
-           * Флаг необязательности поля
-           */
-          optional?: false;
-      }
-    | {
-          /**
-           * Флаг обязательности поля
-           */
-          required?: false;
-          /**
-           * Задает выравнивание индикатора обязательности поля
-           * @default right
-           */
-          requiredPlacement?: never;
-          /**
-           * Флаг необязательности поля
-           */
-          optional?: boolean;
-      };
+export type RequiredProps = {
+    /**
+     * Флаг обязательности поля
+     */
+    required?: boolean;
+    /**
+     * Задает выравнивание индикатора обязательности поля
+     * @default right
+     */
+    requiredPlacement?: 'left' | 'right';
+    /**
+     * Флаг необязательности поля
+     */
+    optional?: boolean;
+};
 
 type ClearProps =
     | {