VKUI icon indicating copy to clipboard operation
VKUI copied to clipboard

feat(Calendar): add doneButtonShow, doneButtonDisabled props flags

Open EldarMuhamethanov opened this issue 1 year ago • 5 comments

  • close #7733

  • [x] Unit-тесты
  • [x] e2e-тесты
  • [x] Release notes

Описание

Необходимо для компонента Calendar добавить возможность скрыть кнопку done, а также возможность ее задизаблить

Release notes

Улучшения

  • Calendar:
    • Добавлено свойство doneButtonShow для скрытия кнопки "done"
    • Добавлено свойство doneButtonDisabled для блокирования кнопки "disabled"

EldarMuhamethanov avatar Oct 10 '24 14:10 EldarMuhamethanov

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% 🔽)

github-actions[bot] avatar Oct 10 '24 14:10 github-actions[bot]

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.

codesandbox-ci[bot] avatar Oct 10 '24 14:10 codesandbox-ci[bot]

e2e tests

⚠️ Some screenshots were failed. See Playwright Report.

Playwright Report

github-actions[bot] avatar Oct 10 '24 14:10 github-actions[bot]

👀 Docs deployed

Commit 2110f86c4a2d605bb80f14b27e5a5e1652bbb375

github-actions[bot] avatar Oct 10 '24 14:10 github-actions[bot]

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.

codecov[bot] avatar Oct 10 '24 14:10 codecov[bot]

@EldarMuhamethanov А что думаешь по поводу рассуждений @Kritolan в https://github.com/VKCOM/VKUI/issues/7733#issuecomment-2411535342

В этом PR мы игнорируем DateInput, а там и правда closeOnChange отключается когда enableTime включен. Мы намеренно не даём это делать в DateInput?

Если подумать, то кнопка "Готово" затем и нужна, чтобы иметь возможность закрыть календарь. Если её намеренно прячут, то остаётся только закрытие по клику снаружи, как если бы closeOnChange было false у календаря без времени.

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

@EldarMuhamethanov А что думаешь по поводу рассуждений @Kritolan в #7733 (comment)

В этом PR мы игнорируем DateInput, а там и правда closeOnChange отключается когда enableTime включен. Мы намеренно не даём это делать в DateInput?

Если подумать, то кнопка "Готово" затем и нужна, чтобы иметь возможность закрыть календарь. Если её намеренно прячут, то остаётся только закрытие по клику снаружи, как если бы closeOnChange было false у календаря без времени.

Возможно, если кто-то хочет скрыть эту кнопку, значит он хочет сделать закрытие и применение по-другому. Возможно обернув календарь во что-то и добавив свою кнопку. Другое дело, такое вряд ли получится сделать в DateInput. Возможно в DateInput стоит убрать возможность скрытия этой кнопки

EldarMuhamethanov avatar Oct 24 '24 11:10 EldarMuhamethanov

❌ Patch

Не удалось автоматически применить исправление на ветке 7.0-stable.

Дальнейшие действия выполняют контрибьютеры из группы @VKCOM/vkui-core

Чтобы изменение попало в ветку 7.0-stable, выполните следующие действия:

  1. Создайте новую ветку от 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
  1. Исправьте конфликты, следуя инструкциям из терминала
  2. Отправьте ветку на 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"

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

❌ Patch

Не удалось автоматически применить исправление на ветке v6.

Дальнейшие действия выполняют контрибьютеры из группы @VKCOM/vkui-core

Чтобы изменение попало в ветку v6, выполните следующие действия:

  1. Создайте новую ветку от 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
  1. Исправьте конфликты, следуя инструкциям из терминала
  2. Отправьте ветку на GitHub и создайте новый PR с веткой v6 (установка лейбла не требуется!)
git push --set-upstream origin patch/pr7736
gh pr create --base v6 --title "patch: pr7736" --body "- patch #7736"

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