implement base button with pressable component #29
Conversation
| type RoundButtonType = 'primary' | 'secondary' | ||
|
|
||
| export type BaseButtonPropsType = { | ||
| isDisabled?: boolean | null |
There was a problem hiding this comment.
For the sake of consistency with the RN Button this prop should be named disabled.
| onPress?: () => void | ||
| iconAtStart?: ReactElement | ||
| iconAtEnd?: ReactElement | ||
| text?: string | ReactElement |
There was a problem hiding this comment.
I see you stopped at the 5th iteration of the Button. Take a look at the problems described further: https://www.smashingmagazine.com/2023/01/key-good-component-design-selfishness/#iteration-6 and the final form: hthttps://www.smashingmagazine.com/2023/01/key-good-component-design-selfishness/#me-myself-and-ui
There was a problem hiding this comment.
I think now done in favour of flexibility
| styleText?: StyleProp<TextStyle> | ||
| rootStyle?: StyleProp<ViewStyle> | ||
| hitSlop?: null | Insets | number | ||
| isLoading?: boolean |
There was a problem hiding this comment.
Let's rename this to loading for the sake of consistency with disabled
template/src/config/ui/colors.ts
Outdated
| ANGRY = '#dd3333', | ||
| DEEP_PURPLE = '#5D2555', | ||
| TRANSPARENT = 'rgba(0, 0, 0, 0)', | ||
| TRANSPARENT_DISABLED_BUTTON = 'rgba(0,0,1,0.14)', |
There was a problem hiding this comment.
Let's not tie the names to how they might be used. Note that we have Palette.ANGRY not Palette.ERROR_MESSAGE.
There was a problem hiding this comment.
Thanks for noticing, renamed
#22