Skip to content
This repository has been archived by the owner on Jul 28, 2024. It is now read-only.

[DS-39] feat: implement time picker #184

Merged
merged 5 commits into from
Aug 13, 2021
Merged

[DS-39] feat: implement time picker #184

merged 5 commits into from
Aug 13, 2021

Conversation

rchubarkin
Copy link
Contributor

No description provided.

Copy link

@risen228 risen228 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Думаю внутри можно обойтись без стейта для значения, посмотрю попозже

src/calendar/time-picker/index.tsx Outdated Show resolved Hide resolved
src/calendar/time-picker/index.tsx Outdated Show resolved Hide resolved
src/calendar/time-picker/index.tsx Outdated Show resolved Hide resolved
src/calendar/time-picker/index.tsx Outdated Show resolved Hide resolved
@risen228
Copy link

risen228 commented Aug 3, 2021

Кстати в названии ветки нет задачи

@risen228
Copy link

risen228 commented Aug 3, 2021

Странная штука, ты знаешь откуда это идет? Похоже оно где-то в gatsby-теме

CleanShot 2021-08-03 at 15 37 13@2x

src/calendar/time-picker/index.tsx Outdated Show resolved Hide resolved
src/calendar/time-picker/index.tsx Outdated Show resolved Hide resolved
src/calendar/time-picker/index.tsx Show resolved Hide resolved
src/calendar/time-picker/usage.mdx Show resolved Hide resolved
@rchubarkin rchubarkin force-pushed the feat/time-picker branch 2 times, most recently from 4d21455 to 739b447 Compare August 6, 2021 15:28
Copy link
Contributor

@ainursharaev ainursharaev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Оставил комментария 👆🏻

@ainursharaev ainursharaev self-requested a review August 13, 2021 08:50
@rchubarkin rchubarkin merged commit aa861f5 into master Aug 13, 2021
@rchubarkin rchubarkin deleted the feat/time-picker branch August 13, 2021 10:46
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
@atoms Changes inside atoms @molecules
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants