VKUI icon indicating copy to clipboard operation
VKUI copied to clipboard

[Feature][Calendar]: Добавить возможность скрытия кнопки `Done`

Open EldarMuhamethanov opened this issue 1 year ago • 3 comments

Описание

Сейчас при прокидывании enableTime в Calendar появляется блок с выбором времени и кнопкой Done. Сейчас в этой кнопке можно поменять только текст через doneButtonText. Нужно добавить возможность скрытия данной кнопки, а также подумать какие еще свойства могут понадобиться для данной кнопки(например disabled). Также при отключении селекты со временем должны центрироваться

Скриншоты

f5WZN_lrqyQaNkZtlJIOa8kX0q79owBXWpDf__y0rXsv7XmPO8foBzbGD_KPxK5hziWcBGQkXrKbZrWYlvtowpUv

EldarMuhamethanov avatar Oct 10 '24 12:10 EldarMuhamethanov

А как закрывать календарь без этой кнопки? После выбора секунд? Или только кликом за пределы календаря? Возможно вместо добавления множества свойств с префиксом имеет смысл передавать объект с настройками? Либо в целом передавать всю кнопку как свойство?

Kritolan avatar Oct 14 '24 14:10 Kritolan

А как закрывать календарь без этой кнопки? После выбора секунд? Или только кликом за пределы календаря? Возможно вместо добавления множества свойств с префиксом имеет смысл передавать объект с настройками? Либо в целом передавать всю кнопку как свойство?

Сама по себе эта кнопка календарь не закрывает, а только вызывает обработчик, который ей передается, так что поведение закрытия календаря можно реализовать по собственному усмотрению.

lorentzimys avatar Oct 14 '24 14:10 lorentzimys

А, ну да, спутал с DateInput. Видимо это будет уже несколько оффтопик, но вопрос все еще актуален т.к. DateInput как я понимаю наследует свойства календаря, а значит в нем тоже можно будет отключить кнопку. Это в целом совпадает с ситуацией когда выключено свойство closeOnChange, но как закрывать календарь, кроме как кликом снаружи непонятно. Тут ксатати еще можно заменить, что свойство closeOnChange не работает, при включенном времени у DateInput.

Kritolan avatar Oct 14 '24 15:10 Kritolan

А, ну да, спутал с DateInput. Видимо это будет уже несколько оффтопик, но вопрос все еще актуален т.к. DateInput как я понимаю наследует свойства календаря, а значит в нем тоже можно будет отключить кнопку. Это в целом совпадает с ситуацией когда выключено свойство closeOnChange, но как закрывать календарь, кроме как кликом снаружи непонятно. Тут ксатати еще можно заменить, что свойство closeOnChange не работает, при включенном времени у DateInput.

Привет @Kritolan

Спасибо, что поделился своими мыслями, заставил задуматься.

Мы намеренно не будет добавлять эту возможность в DateInput, как раз по причинам, которые ты обозначил. DateInput наследует свойства календаря, но не все.

andrey-medvedev-vk avatar Oct 24 '24 12:10 andrey-medvedev-vk

v6.7.4 🎉

vkcom-publisher avatar Nov 13 '24 08:11 vkcom-publisher

v7.0.0-beta.2 🎉

vkcom-publisher avatar Nov 25 '24 17:11 vkcom-publisher

v7.0.0 🎉

vkcom-publisher avatar Dec 02 '24 16:12 vkcom-publisher