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: color_separator_primary dpi #5791

Merged
merged 1 commit into from
Sep 12, 2023

Conversation

SevereCloud
Copy link
Contributor

@SevereCloud SevereCloud commented Sep 12, 2023

  • [ ] Unit-тесты визуальное исправление
  • [ ] e2e-тесты нет инфры под разный dpi
  • [ ] Дизайн-ревью визуальное исправление

Описание

Делаем color_separator_primary в зависимости от dpi

@SevereCloud SevereCloud requested a review from a team as a code owner September 12, 2023 10:41
@github-actions
Copy link
Contributor

github-actions bot commented Sep 12, 2023

size-limit report 📦

Path Size
JS 346.1 KB (0%)
JS (gzip) 103.22 KB (0%)
JS (brotli) 84.33 KB (0%)
JS import Div (tree shaking) 3.04 KB (0%)
CSS 278.32 KB (-0.07% 🔽)
CSS (gzip) 36.27 KB (-0.16% 🔽)
CSS (brotli) 28.73 KB (-0.05% 🔽)

inomdzhon
inomdzhon previously approved these changes Sep 12, 2023
Copy link
Contributor

@inomdzhon inomdzhon left a comment

Choose a reason for hiding this comment

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

лол, забавно) не думал, что цвета тоже в зависимости от DPI шатаются)

@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 3c8edaf:

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 3c8edaf

@codecov
Copy link

codecov bot commented Sep 12, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (a3da40d) 81.99% compared to head (3c8edaf) 81.99%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #5791   +/-   ##
=======================================
  Coverage   81.99%   81.99%           
=======================================
  Files         298      298           
  Lines        9839     9839           
  Branches     3115     3115           
=======================================
  Hits         8067     8067           
  Misses       1772     1772           
Flag Coverage Δ
unittests 81.99% <ø> (ø)

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.

@SevereCloud SevereCloud marked this pull request as draft September 12, 2023 10:50
Исправляем color_separator_primary в зависимости от dpi

В будущем требуется переименовать токены

```diff
- --vkui_internal--color_separator_primary: var(--vkui--color_separator_primary);
+ --vkui--color_separator_primary: var(--vkui--color_separator_primary1x);
```
@SevereCloud SevereCloud force-pushed the SevereCloud/feat/color_separator_primary-dpi branch from 0d53140 to 3c8edaf Compare September 12, 2023 11:10
@SevereCloud SevereCloud marked this pull request as ready for review September 12, 2023 11:28
@SevereCloud SevereCloud self-assigned this Sep 12, 2023
@SevereCloud SevereCloud added this to the v5.9.0 milestone Sep 12, 2023
@SevereCloud SevereCloud merged commit c965048 into master Sep 12, 2023
44 checks passed
@SevereCloud SevereCloud deleted the SevereCloud/feat/color_separator_primary-dpi branch September 12, 2023 19:31
SevereCloud added a commit that referenced this pull request Sep 12, 2023
Исправляем color_separator_primary в зависимости от dpi

В будущем требуется переименовать токены

```diff
- --vkui_internal--color_separator_primary: var(--vkui--color_separator_primary);
+ --vkui--color_separator_primary: var(--vkui--color_separator_primary1x);
```
@SevereCloud SevereCloud mentioned this pull request Sep 12, 2023
SevereCloud added a commit that referenced this pull request Sep 13, 2023
Исправляем color_separator_primary в зависимости от dpi

В будущем требуется переименовать токены

```diff
- --vkui_internal--color_separator_primary: var(--vkui--color_separator_primary);
+ --vkui--color_separator_primary: var(--vkui--color_separator_primary1x);
```
@vkcom-publisher
Copy link
Contributor

v5.9.0 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants