review: fix typo and logic

VKCOM · Dec 4, 2023 · eb01699 · eb01699
1 parent 50628be
commit eb01699
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 7 deletions.
diff --git a/packages/vkui/src/components/ChipsInputBase/Chip/Chip.tsx b/packages/vkui/src/components/ChipsInputBase/Chip/Chip.tsx
@@ -34,8 +34,7 @@ export const Chip = ({
   ...restProps
 }: ChipProps) => {
   const { sizeY = 'none' } = useAdaptivity();
-  // const [focusVisible, setFocusVisible] = React.useState(false);
-  const { focusVisible, onFocus, onBlur } = useFocusVisible(true);
+  const { focusVisible, onFocus, onBlur } = useFocusVisible();
   const focusVisibleClassName = useFocusVisibleClassName({ focusVisible });
   const removeLabel = `${removeAriaLabel} ${getTitleFromChildren(children)}`;
 

diff --git a/packages/vkui/src/components/ChipsSelect/useChipsSelect.ts b/packages/vkui/src/components/ChipsSelect/useChipsSelect.ts
@@ -22,9 +22,9 @@ export interface UseChipsSelectProps<O extends ChipOption = ChipOption>
   extends UseChipsInputProps<O> {
   presets?: O[];
   /**
-   * Возможность создавать чипы которых нет в спискеЖ
-   * - `true` – по кнопки Enter
-   * - `string` – через кнопку в пункте меню.
+   * Возможность создавать чипы которых нет в списке:
+   * - `true` – добавление по кнопке Enter;
+   * - `<текст>` – помимо возможности добавления через Enter, в пункте меню появится кнопка с текстом.
    * Текст для пункта создающего чипы при клике, так же отвечает за то будет ли показан этот пункт (показывается после того как в списке не останется опций).
    */
   creatable?: boolean | string;
@@ -200,8 +200,8 @@ function transformPresets<O extends ChipOption>({
     : presetsProp;
 
   if (filteredPresetsProp.length === 0) {
-    if (inputValue !== DEFAULT_INPUT_VALUE && creatable) {
-      return [{ actionText: typeof creatable === 'string' ? creatable : '' }];
+    if (inputValue !== DEFAULT_INPUT_VALUE && typeof creatable === 'string') {
+      return [{ actionText: creatable }];
     }
     return [{ placeholder: emptyText }];
   }