@true-engineering/true-react-common-ui-kit 4.0.0-alpha6 → 4.0.0-alpha62

package/src/components/Notification/Notification.stories.tsx

@@ -1,51 +1,52 @@
-import { Meta, Story } from '@storybook/react';
-import { iconsList, IIconType } from '../Icon';
-import { complexIcons } from '../Icon/complexIcons';
+import { FC } from 'react';
+import { type Meta } from '@storybook/react';
+import { excludeStorybookParams } from '../../helpers';
+import { iconsMap } from '../Icon';
 import { INotificationProps, Notification } from './Notification';
 
-const icons = [undefined, ...Object.keys(iconsList), ...Object.keys(complexIcons)] as IIconType[];
-
-const lorem = 'Lorem ipsum dolor, sit amet consectetur adipisicing';
-
-interface IStory extends INotificationProps {
+interface INotificationCustomProps extends INotificationProps {
   shouldUseChildrenInstead: boolean;
 }
 
-export default {
+const Story: FC<INotificationCustomProps> = ({
+  shouldUseChildrenInstead,
+  title,
+  text,
+  ...args
+}) => (
+  <Notification
+    {...args}
+    title={!shouldUseChildrenInstead ? title : undefined}
+    text={!shouldUseChildrenInstead ? text : undefined}
+  >
+    {shouldUseChildrenInstead ? (
+      <div>
+        Some childrens with <a href="google.com">tags</a>
+      </div>
+    ) : undefined}
+  </Notification>
+);
+
+const meta: Meta<typeof Story> = {
   title: 'Data Display/Notification',
-  component: Notification,
+  component: Story,
   args: {
     title: 'Notification title',
-    text: lorem,
+    text: 'Lorem ipsum dolor, sit amet consectetur adipisicing',
     size: 's',
     type: 'info',
     isFullWidth: false,
     shouldUseChildrenInstead: false,
   },
   argTypes: {
-    size: { control: 'inline-radio', options: ['s', 'm', 'l'] },
-    icon: { options: icons, control: 'select' },
-    type: {
-      control: 'inline-radio',
-      options: ['error', 'info', 'warning', 'ok', 'not-ok', 'custom'],
+    icon: { options: [undefined, ...Object.keys(iconsMap)], control: 'select' },
+  },
+  parameters: {
+    controls: {
+      exclude: excludeStorybookParams(['children']),
     },
   },
-} as Meta<IStory>;
-
-export const Default: Story<IStory> = ({ title, text, shouldUseChildrenInstead, ...args }) => {
-  const children = (
-    <div>
-      Some childrens with <a href="google.com">tags</a>
-    </div>
-  );
-
-  return (
-    <Notification
-      {...args}
-      title={!shouldUseChildrenInstead ? title : undefined}
-      text={!shouldUseChildrenInstead ? text : undefined}
-    >
-      {shouldUseChildrenInstead ? children : undefined}
-    </Notification>
-  );
 };
+
+export default meta;
+export const Default = {};

package/src/components/Notification/Notification.tsx

@@ -1,11 +1,11 @@
 import { FC, ReactNode } from 'react';
 import clsx from 'clsx';
 import {
+  addDataAttributes,
   addDataTestId,
   isReactNodeNotEmpty,
   isStringNotEmpty,
 } from '@true-engineering/true-react-platform-helpers';
-import { addDataAttributes } from '../../helpers';
 import { ICommonProps } from '../../types';
 import { Icon, IIcon, renderIcon } from '../Icon';
 import { INotificationType } from './types';
@@ -49,8 +49,7 @@ export const Notification: FC<INotificationProps> = ({
         [classes.withTitle]: hasTitle,
         [classes.withText]: hasText,
       })}
-      {...addDataTestId(testId)}
-      {...addDataAttributes(data)}
+      {...addDataAttributes(data, testId)}
     >
       {hasIcon && (
         <div className={classes.icon}>

package/src/components/NumberInput/NumberInput.stories.tsx

@@ -1,33 +1,37 @@
-import { useState } from 'react';
-import { ComponentStory } from '@storybook/react';
-import { NumberInput } from './NumberInput';
+import { FC, useState } from 'react';
+import { type Meta } from '@storybook/react';
+import { INumberInputProps, NumberInput } from './NumberInput';
 
-export default {
-  title: 'Inputs/NumberInput',
-  component: NumberInput,
-};
-
-const Template: ComponentStory<typeof NumberInput> = (args) => {
+const Story: FC<INumberInputProps> = (args) => {
   const [value, setValue] = useState<number>();
-  return <NumberInput {...args} value={value} onChange={(v) => setValue(v)} />;
+  return (
+    <div style={{ width: 300 }}>
+      <NumberInput {...args} value={value} onChange={(v) => setValue(v)} />
+    </div>
+  );
 };
 
-export const Default = Template.bind({});
-
-Default.args = {
-  min: 0,
-  max: 9999,
-  precision: 3,
-  intPartPrecision: 7,
-  canBeFloat: false,
-  canBeNegative: false,
-  label: 'Label',
-  placeholder: 'Placeholder',
-  infoMessage: 'Message Info',
-  units: 'RUB',
-  isInvalid: false,
-  errorMessage: 'Error Text',
-  isDisabled: false,
-  isRequired: false,
-  isReadonly: false,
+const meta: Meta<typeof Story> = {
+  title: 'Inputs/NumberInput',
+  component: Story,
+  args: {
+    min: 0,
+    max: 9999,
+    precision: 3,
+    intPartPrecision: 7,
+    canBeFloat: false,
+    canBeNegative: false,
+    label: 'Label',
+    placeholder: 'Placeholder',
+    infoMessage: 'Message Info',
+    units: 'RUB',
+    isInvalid: false,
+    errorMessage: 'Error Text',
+    isDisabled: false,
+    isRequired: false,
+    isReadonly: false,
+  },
 };
+
+export default meta;
+export const Default = {};

package/src/components/NumberInput/NumberInput.tsx

@@ -28,107 +28,105 @@ export interface INumberInputProps extends Omit<IInputProps, 'value' | 'onChange
   onChange: (value: number | undefined, event?: IChangeInputEvent) => void;
 }
 
-export const NumberInput = forwardRef<HTMLInputElement, INumberInputProps>(
-  (
-    {
-      value,
-      canBeFloat = false,
-      canBeNegative = false,
-      defaultValue,
-      precision = 3,
-      intPartPrecision = 7,
-      onChange,
-      onBlur,
-      min,
-      max,
-      ...props
-    },
-    ref,
-  ) => {
-    const [showedValue, setShowedValue] = useState(() =>
-      formatNumberWithDefault(value, defaultValue),
-    );
+export const NumberInput = forwardRef<HTMLInputElement, INumberInputProps>(function NumberInput(
+  {
+    value,
+    canBeFloat = false,
+    canBeNegative = false,
+    defaultValue,
+    precision = 3,
+    intPartPrecision = 7,
+    onChange,
+    onBlur,
+    min,
+    max,
+    ...props
+  },
+  ref,
+) {
+  const [showedValue, setShowedValue] = useState(() =>
+    formatNumberWithDefault(value, defaultValue),
+  );
 
-    // специальный костыль для обработки Delete - т.к. без него каретка будет работать неправильно при удалении пробелов
-    const [isDeleteForwardAction, setIsDeleteForwardAction] = useState(false);
+  // специальный костыль для обработки Delete - т.к. без него каретка будет работать неправильно при удалении пробелов
+  const [isDeleteForwardAction, setIsDeleteForwardAction] = useState(false);
 
-    const handleKeyDown = (event: KeyboardEvent<HTMLInputElement>) => {
-      setIsDeleteForwardAction(event.key === 'Delete');
-    };
+  const handleKeyDown = (event: KeyboardEvent<HTMLInputElement>) => {
+    setIsDeleteForwardAction(event.key === 'Delete');
+  };
 
-    const setShowedValueWithDefault = (v: string) => {
-      setShowedValue(isStringNotEmpty(v) ? v : formatNumber(defaultValue));
-    };
+  const setShowedValueWithDefault = (v: string) => {
+    setShowedValue(isStringNotEmpty(v) ? v : formatNumber(defaultValue));
+  };
 
-    const handleChange = async (inputValue: string, event: IChangeInputEvent): Promise<void> => {
-      const oldValue = showedValue;
-      const target = event.target as HTMLInputElement;
-      const defaultCaretPos = target.selectionStart ?? 0;
-      const numString = removeStringFormat(inputValue);
+  const handleChange = async (inputValue: string, event: IChangeInputEvent): Promise<void> => {
+    const oldValue = showedValue;
+    const target = event.target as HTMLInputElement;
+    const defaultCaretPos = target.selectionStart ?? 0;
+    const numString = removeStringFormat(inputValue);
 
-      if (numString === '') {
-        onChange(undefined);
-        // не убирать - ломается каретка
-        setShowedValue('');
-        return;
-      }
+    if (numString === '') {
+      onChange(undefined);
+      // не убирать - ломается каретка
+      setShowedValue('');
+      return;
+    }
 
-      const numberRegexp = new RegExp(
-        constructRegExp({
-          precision,
-          intPartPrecision,
-          canBeNegative,
-          canBeFloat,
-        }),
-      );
+    const numberRegexp = new RegExp(
+      constructRegExp({
+        precision,
+        intPartPrecision,
+        canBeNegative,
+        canBeFloat,
+      }),
+    );
 
-      if (!numberRegexp.test(numString)) {
-        await setShowedValue(oldValue);
-        setCaretPosition(target, defaultCaretPos - 1);
-        return;
-      }
+    if (!numberRegexp.test(numString)) {
+      await setShowedValue(oldValue);
+      setCaretPosition(target, defaultCaretPos - 1);
+      return;
+    }
 
-      const hasDot = canBeFloat && /^[-]*[\d\s]*[.,]$/.test(inputValue);
-      const formattedValue = formatStringNumber(numString);
-      const newValue = hasDot ? `${formattedValue}.` : formattedValue;
-      await setShowedValue(newValue);
-      setCaretPosition(
-        target,
-        getNewCaretPosition(defaultCaretPos, oldValue, newValue, isDeleteForwardAction),
-      );
-    };
+    const hasDot = canBeFloat && /^[-]*[\d\s]*[.,]$/.test(inputValue);
+    const formattedValue = formatStringNumber(numString);
+    const newValue = hasDot ? `${formattedValue}.` : formattedValue;
+    await setShowedValue(newValue);
+    setCaretPosition(
+      target,
+      getNewCaretPosition(defaultCaretPos, oldValue, newValue, isDeleteForwardAction),
+    );
+  };
 
-    const handleBlur = (event: FocusEvent<HTMLInputElement>): void => {
-      const val = event.currentTarget.value;
-      const num = stringToNumber(val);
-      const newValue = num !== undefined ? getNumberInRange(num, min, max) : undefined;
-      if (newValue !== value) {
-        onChange(newValue);
-      } else {
-        setShowedValueWithDefault(formatNumber(value));
-      }
-      if (onBlur !== undefined) {
-        onBlur(event);
-      }
-    };
+  const handleBlur = (event: FocusEvent<HTMLInputElement>): void => {
+    const val = event.currentTarget.value;
+    const num = stringToNumber(val);
+    const newValue = num !== undefined ? getNumberInRange(num, min, max) : undefined;
+    if (newValue !== value) {
+      onChange(newValue);
+    } else {
+      setShowedValueWithDefault(formatNumber(value));
+    }
+    if (onBlur !== undefined) {
+      onBlur(event);
+    }
+  };
 
-    useDidMountEffect(() => {
-      if (value !== undefined) {
-        setShowedValueWithDefault(formatNumber(value));
-      } else {
-        setShowedValue(formatNumber(value));
-      }
-    }, [value]);
+  useDidMountEffect(() => {
+    if (value !== undefined) {
+      setShowedValueWithDefault(formatNumber(value));
+    } else {
+      setShowedValue(formatNumber(value));
+    }
+  }, [value]);
 
-    return (
-      <Input
-        value={showedValue}
-        onChange={handleChange}
-        onBlur={handleBlur}
-        onKeyDown={handleKeyDown}
-        ref={ref}
-        {...props}
-      />
-    );
-  },
-);
+  return (
+    <Input
+      value={showedValue}
+      onChange={handleChange}
+      onBlur={handleBlur}
+      onKeyDown={handleKeyDown}
+      ref={ref}
+      {...props}
+    />
+  );
+});

package/src/components/PhoneInput/PhoneInput.stories.tsx

@@ -1,67 +1,41 @@
-import { useState } from 'react';
-import { ComponentStory } from '@storybook/react';
-import { phoneInfo } from '../../constants';
-import { iconsList } from '../Icon';
-import { PhoneInput } from './PhoneInput';
+import { FC, useState } from 'react';
+import { type Meta } from '@storybook/react';
+import { GROUP_PLACEMENTS } from '../ControlWrapper';
+import { iconsMap } from '../Icon';
+import { IPhoneInputProps, PhoneInput } from './PhoneInput';
 import { IPhoneValue } from './types';
 
-const icons = [undefined, ...Object.keys(iconsList)];
-const groupPlacements = [undefined, 'left', 'right', 'middle'];
-const borders = [undefined, 'left', 'top', 'right', 'bottom'];
-const errorPositions = ['bottom', 'top'];
-const countryCodes = phoneInfo.map((info) => info.countryCode);
-
-export default {
-  title: 'Inputs/PhoneInput',
-  component: PhoneInput,
-  argTypes: {
-    icon: { control: 'select', options: icons },
-    errorPosition: { control: 'inline-radio', options: errorPositions },
-    border: { control: 'inline-radio', options: borders },
-    groupPlacement: { control: 'select', options: groupPlacements },
-    defaultCountryCode: { control: 'select', options: countryCodes },
-  },
-};
-
-const Template: ComponentStory<typeof PhoneInput> = (args) => {
+const Story: FC<IPhoneInputProps> = (args) => {
   const [value, setValue] = useState<IPhoneValue>({
     dialCode: '7',
     phoneNumber: '9134567890',
     countryCode: 'RU',
   });
-
   return <PhoneInput {...args} value={value} onChange={setValue} />;
 };
 
-export const Default = Template.bind({});
-
-Default.args = {
-  label: 'Label',
-  infoMessage: 'Message Info',
-  icon: undefined,
-  isInvalid: false,
-  errorMessage: 'Error Text',
-  isActive: false,
-  isDisabled: false,
-  isRequired: false,
-  isReadonly: false,
-  isClearable: false,
-  isLoading: false,
-  groupPlacement: undefined,
-};
-
-Default.parameters = {
-  controls: {
-    exclude: [
-      'testId',
-      'data',
-      'tabIndex',
-      'onChange',
-      'onFocus',
-      'onBlur',
-      'onPaste',
-      'onKeyDown',
-      'onIconClick',
-    ],
+const meta: Meta<typeof Story> = {
+  title: 'Inputs/PhoneInput',
+  component: Story,
+  args: {
+    label: 'Label',
+    infoMessage: 'Message Info',
+    endIcon: undefined,
+    isInvalid: false,
+    errorMessage: 'Error Text',
+    isActive: false,
+    isDisabled: false,
+    isRequired: false,
+    isReadonly: false,
+    isClearable: false,
+    isLoading: false,
+    groupPlacement: undefined,
+  },
+  argTypes: {
+    endIcon: { control: 'select', options: [undefined, ...Object.keys(iconsMap)] },
+    groupPlacement: { options: [undefined, ...GROUP_PLACEMENTS], control: 'select' },
   },
 };
+
+export default meta;
+export const Default = {};

package/src/components/PhoneInput/PhoneInput.tsx

@@ -1,8 +1,7 @@
 import { FC, FocusEvent, useMemo, useRef, useState } from 'react';
 import clsx from 'clsx';
-import { addDataTestId, getTestId } from '@true-engineering/true-react-platform-helpers';
+import { addDataAttributes, getTestId } from '@true-engineering/true-react-platform-helpers';
 import {
-  addDataAttributes,
   findCountryByCode,
   getCountryCodeFromPhone,
   getFullPhone,
@@ -76,7 +75,7 @@ export const PhoneInput: FC<IPhoneInputProps> = ({
 
   const countryCode = useMemo(
     () => value?.countryCode ?? getCountryCodeFromPhone(phoneWithCode),
-    [value.countryCode, value.dialCode, value.phoneNumber],
+    [phoneWithCode, value?.countryCode],
   );
 
   const handleClose = () => {
@@ -157,7 +156,7 @@ export const PhoneInput: FC<IPhoneInputProps> = ({
   useOnClickOutsideWithRef(list, handleClose, inputWrapper);
 
   return (
-    <div className={classes.root} {...addDataTestId(testId)} {...addDataAttributes(data)}>
+    <div className={classes.root} {...addDataAttributes(data, testId)}>
       <div
         className={clsx(classes.countrySelectContainer, {
           [classes.disabledCountrySelect]: isDisabled,

package/src/components/PhoneInput/components/PhoneInputCountryList/PhoneInputCountryList.stories.tsx

@@ -1,7 +1,7 @@
-import { ComponentStory, ComponentMeta } from '@storybook/react';
+import { type Meta } from '@storybook/react';
 import { PhoneInputCountryList } from './PhoneInputCountryList';
 
-export default {
+const meta: Meta<typeof PhoneInputCountryList> = {
   title: 'Inputs/PhoneInput/PhoneInputCountryList',
   component: PhoneInputCountryList,
   args: {
@@ -9,13 +9,7 @@ export default {
     placeholder: 'Поиск',
     noMatchesLabel: 'Ничего не найдено',
   },
-  parameters: {
-    controls: {
-      exclude: ['data', 'tweakStyles', 'testId'],
-    },
-  },
-} as ComponentMeta<typeof PhoneInputCountryList>;
+};
 
-export const Default: ComponentStory<typeof PhoneInputCountryList> = (args) => (
-  <PhoneInputCountryList {...args} />
-);
+export default meta;
+export const Default = {};

package/src/components/PhoneInput/components/PhoneInputCountryList/PhoneInputCountryList.tsx

@@ -48,10 +48,7 @@ export const PhoneInputCountryList: FC<IPhoneInputCountryListProps> = ({
     currentComponentName: 'PhoneInputCountryList',
   });
 
-  const sortedCountries = useMemo(
-    () => sortCountriesByLocale(phoneInfo, locale),
-    [phoneInfo, locale],
-  );
+  const sortedCountries = useMemo(() => sortCountriesByLocale(phoneInfo, locale), [locale]);
 
   const selectedCountryIdx = findCountryIndexByCode(selectedCountryCode, sortedCountries);
 

package/src/components/RadioButton/RadioButton.stories.tsx

@@ -1,15 +1,18 @@
-import { useState } from 'react';
-import { ComponentStory } from '@storybook/react';
-import { RadioButton } from './RadioButton';
-
-export default {
-  title: 'Controls/RadioButton',
-  component: RadioButton,
-};
+import { FC, useState } from 'react';
+import { type Meta } from '@storybook/react';
+import { excludeStorybookParams } from '../../helpers';
+import { IExtendableProps } from '../../types';
+import { IRadioButtonProps, RadioButton } from './RadioButton';
 
 const options = ['one', 'two', 'three'];
+const SIZES = [undefined, 'micro'] as const;
 
-const Template: ComponentStory<typeof RadioButton> = (args) => {
+declare module './types' {
+  // eslint-disable-next-line @typescript-eslint/no-empty-object-type
+  export interface IRadioButtonSizes extends IExtendableProps<typeof SIZES> {}
+}
+
+const Story: FC<IRadioButtonProps<string>> = (args) => {
   const [activeIdx, setActiveIdx] = useState(0);
 
   return (
@@ -30,17 +33,24 @@ const Template: ComponentStory<typeof RadioButton> = (args) => {
   );
 };
 
-export const Default = Template.bind({});
-
-Default.args = {
-  children: 'Label',
-  // isChecked: false,
-  isInvalid: false,
-  isDisabled: false,
-};
-
-Default.parameters = {
-  controls: {
-    exclude: ['data', 'value', 'groupName', 'isChecked', 'onChange'],
+const meta: Meta<typeof Story> = {
+  title: 'Controls/RadioButton',
+  component: Story,
+  args: {
+    children: 'Label',
+    size: undefined,
+    isInvalid: false,
+    isDisabled: false,
+  },
+  argTypes: {
+    size: { options: [undefined, ...SIZES], control: 'inline-radio' },
+  },
+  parameters: {
+    controls: {
+      exclude: excludeStorybookParams(['value', 'groupName', 'isChecked']),
+    },
   },
 };
+
+export default meta;
+export const Default = {};

package/src/components/RadioButton/RadioButton.styles.ts

@@ -1,4 +1,5 @@
 import { ITweakStyles, createThemedStyles } from '../../theme';
+import { IRadioButtonSizes } from './types';
 
 export const useStyles = createThemedStyles('RadioButton', {
   input: {
@@ -34,4 +35,4 @@ export const useStyles = createThemedStyles('RadioButton', {
   isInvalid: {},
 });
 
-export type IRadioButtonStyles = ITweakStyles<typeof useStyles>;
+export type IRadioButtonStyles = ITweakStyles<typeof useStyles, IRadioButtonSizes>;

package/src/components/RadioButton/RadioButton.tsx

@@ -1,14 +1,19 @@
 import { ReactNode } from 'react';
 import clsx from 'clsx';
-import { addDataTestId, isReactNodeNotEmpty } from '@true-engineering/true-react-platform-helpers';
-import { addDataAttributes } from '../../helpers';
+import {
+  isNotEmpty,
+  isReactNodeNotEmpty,
+  addDataAttributes,
+} from '@true-engineering/true-react-platform-helpers';
 import { ICommonProps } from '../../types';
+import { IRadioButtonSize } from './types';
 import { useStyles, IRadioButtonStyles } from './RadioButton.styles';
 
 export interface IRadioButtonProps<Value extends string> extends ICommonProps<IRadioButtonStyles> {
   children?: ReactNode;
   value: Value;
   groupName: string;
+  size?: IRadioButtonSize;
   isChecked?: boolean;
   /** @default false */
   isDisabled?: boolean;
@@ -21,6 +26,7 @@ export function RadioButton<Value extends string>({
   children,
   value,
   groupName,
+  size,
   isChecked,
   isDisabled = false,
   isInvalid = false,
@@ -33,10 +39,11 @@ export function RadioButton<Value extends string>({
 
   return (
     <label
-      className={clsx(classes.label, isDisabled && classes.isDisabled)}
+      className={clsx(classes.label, isNotEmpty(size) && classes[size], {
+        [classes.isDisabled]: isDisabled,
+      })}
       htmlFor={`${groupName}--${value}`}
-      {...addDataTestId(testId)}
-      {...addDataAttributes(data)}
+      {...addDataAttributes(data, testId)}
     >
       <input
         id={`${groupName}--${value}`}