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.

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

feat(CustomSelect,ChipsSelect): add noMaxHeight prop #6045

Merged
merged 2 commits into from
Oct 30, 2023

Conversation

BlackySoul
Copy link
Contributor

@BlackySoul BlackySoul commented Oct 24, 2023


- [ ] Unit-тесты

  • e2e-тесты

- [ ] Дизайн-ревью

- [ ] Документация фичи

Описание

Иногда требуется показать контент без скролла

Изменения

Добавлен проп noMaxHeight, позволяющий отключить задание максимальной высоты

@BlackySoul BlackySoul requested a review from a team as a code owner October 24, 2023 15:23
@github-actions github-actions bot added the v5 Автоматизация: PR продублируется в ветку v5 label Oct 24, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Oct 24, 2023

size-limit report 📦

Path Size
JS 331.69 KB (+0.05% 🔺)
JS (gzip) 101.36 KB (+0.06% 🔺)
JS (brotli) 83.6 KB (+0.04% 🔺)
JS import Div (tree shaking) 2.71 KB (0%)
CSS 255.17 KB (+0.02% 🔺)
CSS (gzip) 33.56 KB (+0.03% 🔺)
CSS (brotli) 27.23 KB (+0.05% 🔺)

@codesandbox-ci
Copy link

codesandbox-ci bot commented Oct 24, 2023

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.

Latest deployment of this branch, based on commit 1adacae:

Sandbox Source
VKUI TypeScript Configuration

@github-actions
Copy link
Contributor

github-actions bot commented Oct 24, 2023

e2e tests

Playwright Report

@github-actions
Copy link
Contributor

github-actions bot commented Oct 24, 2023

👀 Docs deployed

Commit 1adacae

@codecov
Copy link

codecov bot commented Oct 24, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (7ef70b5) 79.21% compared to head (1adacae) 79.23%.
Report is 2 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #6045      +/-   ##
==========================================
+ Coverage   79.21%   79.23%   +0.02%     
==========================================
  Files         305      305              
  Lines        9588     9600      +12     
  Branches     3244     3249       +5     
==========================================
+ Hits         7595     7607      +12     
  Misses       1993     1993              
Flag Coverage Δ
unittests 79.23% <100.00%> (+0.02%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
...es/vkui/src/components/ChipsSelect/ChipsSelect.tsx 83.75% <100.00%> (+0.10%) ⬆️
.../vkui/src/components/CustomSelect/CustomSelect.tsx 86.36% <100.00%> (+0.04%) ⬆️
...ents/CustomSelectDropdown/CustomSelectDropdown.tsx 100.00% <100.00%> (ø)

... and 2 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

SevereCloud
SevereCloud previously approved these changes Oct 24, 2023
inomdzhon
inomdzhon previously approved these changes Oct 24, 2023
@BlackySoul BlackySoul dismissed stale reviews from inomdzhon and SevereCloud via 1adacae October 25, 2023 16:06
@BlackySoul BlackySoul force-pushed the feat/5915/CustomSelectSelect_add_maxWidth_prop branch from 5ddbb02 to 1adacae Compare October 25, 2023 16:06
@BlackySoul BlackySoul merged commit 74b5848 into master Oct 30, 2023
@BlackySoul BlackySoul deleted the feat/5915/CustomSelectSelect_add_maxWidth_prop branch October 30, 2023 09:53
vkcom-publisher pushed a commit that referenced this pull request Oct 30, 2023
* feat(CustomSelect,ChipsSelect): add noMaxHeight prop

* chore: add screenshots
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
v5 Автоматизация: PR продублируется в ветку v5
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature]: В элементе Select/CustomSelect показывать options без скролла
3 participants