Skip to content

fix(datepicker): no input focus after close #124

Merged
merged 4 commits into from
May 30, 2019
Merged

fix(datepicker): no input focus after close #124

merged 4 commits into from
May 30, 2019

Conversation

roll314
Copy link
Contributor

@roll314 roll314 commented May 15, 2019

@fost
@lskramarov
@mikeozornin, я сделал безусловный фокусе вне зависимости от того был ли инпут в фокусе перед открытием datepicker. Так же инпут будут в фокусе после закрытия даже если datepicker был открыт "снаружи" (из другого места или компонента) и изначально он был не в фокусе

@pimenovoleg pimenovoleg requested a review from mikeozornin May 15, 2019 16:57
@pimenovoleg
Copy link
Member

@roll314
см - unit test

@roll314
Copy link
Contributor Author

roll314 commented May 16, 2019

убрал ненужный теперь код в тесте и добавлен новый тест для этого случая

@roll314
Copy link
Contributor Author

roll314 commented May 17, 2019

после обновления messageformat пропал файл messageformat.min.js

tests/karma.conf.js Outdated Show resolved Hide resolved
@pimenovoleg
Copy link
Member

@mikeozornin посмотри - все ок ?

Copy link
Contributor

@mikeozornin mikeozornin left a comment

Choose a reason for hiding this comment

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

Гуди-гуди

@roll314
Copy link
Contributor Author

roll314 commented May 30, 2019

Меджик гуди?

@pimenovoleg pimenovoleg merged commit c9036a1 into positive-js:master May 30, 2019
@pimenovoleg
Copy link
Member

pimenovoleg commented May 30, 2019

Closes #111

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants