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

fix(WriteBar): icons shift #5836

Merged
merged 1 commit into from
Sep 25, 2023
Merged

Conversation

BlackySoul
Copy link
Contributor

@BlackySoul BlackySoul commented Sep 20, 2023

- [ ] Unit-тесты

  • e2e-тесты

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

Описание

В iOS "прыгала" after часть в случае, когда иконка и кнопка располагались вместе, а должны оставаться на своих местах

2023-09-20.11.12.28.mov

Изменения

2023-09-20.11.19.30.mov

Добавила компенсацию отступа для случая, когда в after части находится и кнопка, и иконки вместе.

И исправила обводку в IOS компонента Counter, чтобы она совпадала с фоном (пришлось ещё потрогать threshold, чтобы изменения на скриншотах появились)

@BlackySoul BlackySoul requested a review from a team as a code owner September 20, 2023 04:19
@codesandbox-ci
Copy link

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 035ac91:

Sandbox Source
VKUI TypeScript Configuration

@github-actions
Copy link
Contributor

size-limit report 📦

Path Size
JS 317.37 KB (0%)
JS (gzip) 96.78 KB (0%)
JS (brotli) 80.07 KB (0%)
JS import Div (tree shaking) 2.95 KB (0%)
CSS 275.82 KB (+0.05% 🔺)
CSS (gzip) 36 KB (+0.05% 🔺)
CSS (brotli) 28.5 KB (-0.14% 🔽)

@github-actions
Copy link
Contributor

e2e tests

Playwright Report

@codecov
Copy link

codecov bot commented Sep 20, 2023

Codecov Report

Patch coverage has no change and project coverage change: +0.26% 🎉

Comparison is base (de2126d) 80.87% compared to head (035ac91) 81.13%.
Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #5836      +/-   ##
==========================================
+ Coverage   80.87%   81.13%   +0.26%     
==========================================
  Files         300      300              
  Lines        9166     9193      +27     
  Branches     3122     3125       +3     
==========================================
+ Hits         7413     7459      +46     
+ Misses       1753     1734      -19     
Flag Coverage Δ
unittests 81.13% <ø> (+0.26%) ⬆️

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

see 9 files with indirect coverage changes

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

@github-actions
Copy link
Contributor

👀 Docs deployed

Commit 035ac91

Copy link
Contributor

@mendrew mendrew left a comment

Choose a reason for hiding this comment

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

👍

@SevereCloud SevereCloud added ci:cherry-pick:patch Автоматизация: PR продублируется в ветку последнего минорного релиза для выпуска патча v5 Автоматизация: PR продублируется в ветку v5 labels Sep 22, 2023
@BlackySoul BlackySoul merged commit 460cade into master Sep 25, 2023
@BlackySoul BlackySoul deleted the blackysoul/fix_writebar_shift branch September 25, 2023 10:47
vkcom-publisher pushed a commit that referenced this pull request Sep 25, 2023
vkcom-publisher pushed a commit that referenced this pull request Sep 25, 2023
@BlackySoul BlackySoul added this to the v5.8.2 milestone Sep 25, 2023
@BlackySoul BlackySoul self-assigned this Sep 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci:cherry-pick:patch Автоматизация: PR продублируется в ветку последнего минорного релиза для выпуска патча v5 Автоматизация: PR продублируется в ветку v5
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants