-
Notifications
You must be signed in to change notification settings - Fork 2
Conversation
src/ui/atoms/checkbox/index.tsx
Outdated
onChange(event: React.SyntheticEvent): void; | ||
} | ||
|
||
export const Checkbox: React.FC<CheckboxProps & { className: string }> = ({ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
export const Checkbox: React.FC<CheckboxProps & { className: string }> = ({ | |
const Checkbox: React.FC<CheckboxProps & { className: string }> = ({ |
Думаю, что пользователю не нужен доступ к базовому компоненту без стилей.
Иначе пользователь получит доступ к checkbox.Checkbox
и может быть удивлен, почему это верстка расползлась
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Удалишь export
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@sergeysova сорян, забыла удалить старый Checkbox, компонент лежит в molecules/toggle
1f2aeda
to
756bc4b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Думаю, лучше удалить export
756bc4b
to
1cff1b6
Compare
1cff1b6
to
f226fb3
Compare
Implement switch atom