feat(Calendar): add doneButtonShow, doneButtonDisabled props flags
- close #7733
- [x] Unit-тесты
- [x] e2e-тесты
- [x] Release notes
Описание
Необходимо для компонента Calendar добавить возможность скрыть кнопку done, а также возможность ее задизаблить
Release notes
Улучшения
- Calendar:
- Добавлено свойство
doneButtonShowдля скрытия кнопки"done" - Добавлено свойство
doneButtonDisabledдля блокирования кнопки"disabled"
- Добавлено свойство
size-limit report 📦
| Path | Size |
|---|---|
| JS | 383.27 KB (+0.06% 🔺) |
| JS (gzip) | 122.62 KB (+0.06% 🔺) |
| JS (brotli) | 101.55 KB (+0.03% 🔺) |
| JS import Div (tree shaking) | 1.47 KB (0%) |
| CSS | 342.26 KB (+0.02% 🔺) |
| CSS (gzip) | 49.34 KB (+0.05% 🔺) |
| CSS (brotli) | 39.93 KB (-0.04% 🔽) |
This pull request is automatically built and testable in CodeSandbox.
To see build info of the built libraries, click here or the icon next to each commit SHA.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 95.34%. Comparing base (
f198947) to head (2110f86). Report is 5 commits behind head on master.
Additional details and impacted files
@@ Coverage Diff @@
## master #7736 +/- ##
=======================================
Coverage 95.34% 95.34%
=======================================
Files 377 377
Lines 11054 11059 +5
Branches 3673 3677 +4
=======================================
+ Hits 10539 10544 +5
Misses 515 515
| Flag | Coverage Δ | |
|---|---|---|
| unittests | 95.34% <100.00%> (+<0.01%) |
:arrow_up: |
Flags with carried forward coverage won't be shown. Click here to find out more.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
@EldarMuhamethanov А что думаешь по поводу рассуждений @Kritolan в https://github.com/VKCOM/VKUI/issues/7733#issuecomment-2411535342
В этом PR мы игнорируем DateInput, а там и правда closeOnChange отключается когда enableTime включен.
Мы намеренно не даём это делать в DateInput?
Если подумать, то кнопка "Готово" затем и нужна, чтобы иметь возможность закрыть календарь. Если её намеренно прячут, то остаётся только закрытие по клику снаружи, как если бы closeOnChange было false у календаря без времени.
@EldarMuhamethanov А что думаешь по поводу рассуждений @Kritolan в #7733 (comment)
В этом PR мы игнорируем
DateInput, а там и правдаcloseOnChangeотключается когдаenableTimeвключен. Мы намеренно не даём это делать вDateInput?Если подумать, то кнопка "Готово" затем и нужна, чтобы иметь возможность закрыть календарь. Если её намеренно прячут, то остаётся только закрытие по клику снаружи, как если бы
closeOnChangeбыло false у календаря без времени.
Возможно, если кто-то хочет скрыть эту кнопку, значит он хочет сделать закрытие и применение по-другому. Возможно обернув календарь во что-то и добавив свою кнопку. Другое дело, такое вряд ли получится сделать в DateInput. Возможно в DateInput стоит убрать возможность скрытия этой кнопки
❌ Patch
Не удалось автоматически применить исправление на ветке 7.0-stable.
Дальнейшие действия выполняют контрибьютеры из группы @VKCOM/vkui-core
Чтобы изменение попало в ветку 7.0-stable, выполните следующие действия:
- Создайте новую ветку от 7.0-stable и примените изменения используя cherry-pick
git stash # опционально
git fetch origin 7.0-stable
git checkout -b patch/pr7736 origin/7.0-stable
git cherry-pick --no-commit 5775640781504371c739f6c370bdbc903ebf0c86
git checkout HEAD **/__image_snapshots__/*.png
git diff --quiet HEAD || git commit --no-verify --no-edit
- Исправьте конфликты, следуя инструкциям из терминала
- Отправьте ветку на GitHub и создайте новый PR с веткой 7.0-stable (установка лейбла не требуется!)
git push --set-upstream origin patch/pr7736
gh pr create --base 7.0-stable --title "patch: pr7736" --body "- patch #7736"
❌ Patch
Не удалось автоматически применить исправление на ветке v6.
Дальнейшие действия выполняют контрибьютеры из группы @VKCOM/vkui-core
Чтобы изменение попало в ветку v6, выполните следующие действия:
- Создайте новую ветку от v6 и примените изменения используя cherry-pick
git stash # опционально
git fetch origin v6
git checkout -b patch/pr7736 origin/v6
git cherry-pick --no-commit 5775640781504371c739f6c370bdbc903ebf0c86
git checkout HEAD **/__image_snapshots__/*.png
git diff --quiet HEAD || git commit --no-verify --no-edit
- Исправьте конфликты, следуя инструкциям из терминала
- Отправьте ветку на GitHub и создайте новый PR с веткой v6 (установка лейбла не требуется!)
git push --set-upstream origin patch/pr7736
gh pr create --base v6 --title "patch: pr7736" --body "- patch #7736"