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(Search): add Find button #5790

Merged
merged 7 commits into from
Nov 20, 2023
Merged

Conversation

BlackySoul
Copy link
Contributor

@BlackySoul BlackySoul commented Sep 12, 2023

Описание

Добавляем кнопку "Найти" в соответствии с требованиями дизайн-системы

@github-actions
Copy link
Contributor

github-actions bot commented Sep 12, 2023

size-limit report 📦

Path Size
JS 336.79 KB (+0.12% 🔺)
JS (gzip) 102.88 KB (+0.14% 🔺)
JS (brotli) 84.89 KB (+0.17% 🔺)
JS import Div (tree shaking) 2.71 KB (0%)
CSS 255.61 KB (+0.15% 🔺)
CSS (gzip) 33.63 KB (+0.19% 🔺)
CSS (brotli) 27.29 KB (+0.12% 🔺)

@codesandbox-ci
Copy link

codesandbox-ci bot commented Sep 12, 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 26be94a:

Sandbox Source
VKUI TypeScript Configuration

@github-actions
Copy link
Contributor

github-actions bot commented Sep 12, 2023

e2e tests

Playwright Report

@github-actions
Copy link
Contributor

github-actions bot commented Sep 12, 2023

👀 Docs deployed

Commit 26be94a

@codecov
Copy link

codecov bot commented Sep 12, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (69fc67f) 79.38% compared to head (26be94a) 79.40%.
Report is 2 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #5790      +/-   ##
==========================================
+ Coverage   79.38%   79.40%   +0.01%     
==========================================
  Files         306      306              
  Lines        9631     9639       +8     
  Branches     3261     3265       +4     
==========================================
+ Hits         7646     7654       +8     
  Misses       1985     1985              
Flag Coverage Δ
unittests 79.40% <100.00%> (+0.01%) ⬆️

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

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

@BlackySoul BlackySoul requested a review from a team September 12, 2023 10:13
Zaycevq
Zaycevq previously approved these changes Sep 12, 2023
Copy link

@Zaycevq Zaycevq left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍🏽

@github-actions github-actions bot added the pr-needs-work Автоматизация: PR автоматически закроется через 14 дней при отсутствии активности label Sep 26, 2023
@BlackySoul BlackySoul added no-stale Добавляет PR в исключения для автоматического закрытия and removed pr-needs-work Автоматизация: PR автоматически закроется через 14 дней при отсутствии активности labels Sep 28, 2023
@BlackySoul BlackySoul force-pushed the feat/5521/Search_add_find_button branch from 8be7488 to 26be94a Compare November 16, 2023 12:09
@github-actions github-actions bot added the v5 Автоматизация: PR продублируется в ветку v5 label Nov 16, 2023
@BlackySoul BlackySoul merged commit 8d0e56b into master Nov 20, 2023
@BlackySoul BlackySoul deleted the feat/5521/Search_add_find_button branch November 20, 2023 12:35
vkcom-publisher pushed a commit that referenced this pull request Nov 20, 2023
* feat(Search): add Find button

* fix(Search): remove absolute position + code style fixes

* fix(Search): fix test get buttons func

* feat(Search): mv appear/disappear logic of controls to CSS

* fix(Search): fix flicking on Safari

* feat(Search): add find button

* chore: update screenshots

---------

Co-authored-by: Inomdzhon Mirdzhamolov <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-stale Добавляет PR в исключения для автоматического закрытия v5 Автоматизация: PR продублируется в ветку v5
Projects
None yet
4 participants