Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug]: Исправить типизацию у компонентов, использующих Tappable #3430

Open
Tracked by #5834
SevereCloud opened this issue Oct 11, 2022 · 0 comments

Comments

@SevereCloud
Copy link
Contributor

SevereCloud commented Oct 11, 2022

Описание

Необходимо пройтись по компонентам использующие Tappable и исправить свойства. В компоненте бывают свойства, которые перебиваются в Tappable

export interface CmpProps extends TappableProps {
}

const Cmp = (props:CmpProps) => {
  return (
    <Tappable
        {...props}
        className={styles["Cmp__tappable"]}
        disabled={false}
      />
  )
}

В интерфейсе CmpProps будет доступны свойства className, но оно будет перебито.

Здесь несколько вариантов:

  • убрать свойство из типизации Omit<TappableProps, "className" | "disabled">
  • сделать свойству значение по умолчанию, которое можно перебить
  • другое поведение для слияния свойств(к примеру использовать className={classNamesString(...)} )

По хорошему необходимо проверить все компоненты, которые используют расширения от другого типа.


Также необходимо добавить элементам больше типизации(HorizontalCell, Checkbox и т.д.)

@SevereCloud SevereCloud added this to VKUI Oct 11, 2022
@SevereCloud SevereCloud moved this to 🗃 Backlog in VKUI Oct 11, 2022
@eugpoloz eugpoloz changed the title [Bug]: Исправить типизацию у элементов использующие Tappable [Bug]: Исправить типизацию у компонентов, использующих Tappable Jan 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: 🗃 Backlog
Development

No branches or pull requests

2 participants