[FRNT-570] Implement DatePicker #180
Conversation
tatinacher
commented
Jul 21, 2021
tatinacher
commented
- Write description and API
- Start implementing DatePicker component
|
По этому ПР прошу поглядеть описание, API и возможную реализацию (на данный момент компонент не реализован полностью), докинуть если чего не хватает. |
tatinacher
requested review from
Irinaristova,
ainursharaev,
rchubarkin,
risenxxx and
sergeysova
| "dependencies": { | ||
| "@use-date-input/core": "^0.2.1-beta.0", | ||
| "@use-date-input/dayjs-adapter": "^0.2.0-beta.0", | ||
| "dayjs": "^1.10.6", |
| format = 'DD.MM.YYYY', | ||
| icon, | ||
| initialDate = dayjs().format('DD.MM.YYYY'), | ||
| isOpen = false, |
There was a problem hiding this comment.
suggestion: Если это передается просто без хендлера, стоит наверное назвать как-то вроде defaultOpen / initialOpen, или добавить хендлер по типу onOpenChange. Мб еще можно в зависимости от того, передали снаружи isOpen и onOpenChange, использовать или их, или локальный isOpen
ainursharaev
left a comment
ainursharaev
left a comment
There was a problem hiding this comment.
Посмотри комментарии @risenforces
tatinacher
force-pushed
the
feat/FRNT-570-implement-date-picker
branch
2 times, most recently
from
4461068 to
e5d4f70
Compare
| @@ -0,0 +1 @@ | |||
| export { DatePicker } from './date-picker'; | |||
There was a problem hiding this comment.
question(blocking): Должен ли он находиться в этом пакете? Я думаю что стоит его переместить в пакет calendar, вынес этот вопрос на обсуждение
| @@ -0,0 +1,165 @@ | |||
| import 'react-calendar/dist/Calendar.css'; | |||
There was a problem hiding this comment.
в пакете calendar должно лежать)
| isDayDisabled?: ((props: CalendarTileProperties) => boolean) | undefined; | ||
| isOpen?: boolean; | ||
| locale?: string; | ||
| maxDate?: dayjs.Dayjs; |
There was a problem hiding this comment.
Question: Может обычный Date принимать во всех пропах или строку? А то получается что пользователю нужен пикер, а ему еще dayjs для этого импортировать, при том что он скорее нам внутри пикера нужен чем ему
sergeysova
requested review from
risenxxx
and removed request for
Irinaristova
tatinacher
force-pushed
the
feat/FRNT-570-implement-date-picker
branch
from
e5d4f70 to
d50c7d1
Compare
