Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Feature]: DateInput добавить возможность прокидивать коллбэк на onClose #7865

Closed
andrey-medvedev-vk opened this issue Oct 29, 2024 · 3 comments · Fixed by #7878
Closed

[Feature]: DateInput добавить возможность прокидивать коллбэк на onClose #7865

andrey-medvedev-vk opened this issue Oct 29, 2024 · 3 comments · Fixed by #7878
Assignees
@EldarMuhamethanov
Labels
cmp:date-input cmp:date-range-input type:feature
Milestone
v7.0.0

Comments

@andrey-medvedev-vk
Copy link
Contributor

Описание

На данные момент DateInput в режиме enableTime закрывается не только при клике снаружи, но и при клике на кнопку "готово", а каждое изменение даты, времени, вызывает onChange.
Пользователи DateInput которым требуется вызывать запросы при изменении времени сталкиваются с проблемой, что при каждом клике в календаре у них вызываются лишние запросы.
Хотелось бы для оптимизации иметь возможность завязываться на onClose и вызывать запрос при закрытии календаря.

  • onClose должен вызываться и при клике на "готово" и при клике снаружи.
  • и в режиме closeOnChange = false.

Актуально и для DateRangeInput, так как там есть closeOnChange.

Скриншоты

No response
@github-project-automation github-project-automation bot moved this to 🗃 Backlog in VKUI Oct 29, 2024
@andrey-medvedev-vk andrey-medvedev-vk changed the title [Feature]: DateInput добавить возможность прокидивыть коллбэк на onClose [Feature]: DateInput добавить возможность прокидивать коллбэк на onClose Oct 29, 2024
@andrey-medvedev-vk andrey-medvedev-vk moved this from 🗃 Backlog to 🔜 To do in VKUI Oct 29, 2024
@andrey-medvedev-vk andrey-medvedev-vk mentioned this issue Oct 29, 2024
Closed
@EldarMuhamethanov EldarMuhamethanov self-assigned this Nov 1, 2024
@EldarMuhamethanov EldarMuhamethanov mentioned this issue Nov 1, 2024
Merged
2 tasks
@EldarMuhamethanov EldarMuhamethanov added this to the v6.7.4 milestone Nov 1, 2024
@github-project-automation github-project-automation bot moved this from 👀 In Review to ✅ Done in VKUI Nov 5, 2024
@vkcom-publisher
Copy link
Contributor

✅ v6.7.4 🎉

@vkcom-publisher
Copy link
Contributor

✅ v7.0.0-beta.2 🎉

@inomdzhon inomdzhon modified the milestones: v7.0.0-beta.2, v7.0.0 Dec 2, 2024
@vkcom-publisher
Copy link
Contributor

✅ v7.0.0 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@EldarMuhamethanov EldarMuhamethanov

Labels
cmp:date-input cmp:date-range-input type:feature
Projects
VKUI
Archived in project
Milestone
v7.0.0
Development

Successfully merging a pull request may close this issue.

feat(DateInput, DateRangeInput): add prop onCalendarOpenChanged VKCOM/VKUI
4 participants
@inomdzhon @EldarMuhamethanov @vkcom-publisher @andrey-medvedev-vk