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(Panel): Allow to use gradient in Panel background #6736

Merged

Conversation

mendrew
Copy link
Contributor

@mendrew mendrew commented Mar 22, 2024


  • e2e-тесты

Описание

  • Чтобы была возможность применить градиент к фону Panel через токен используем свойство background вместо background-color.
  • Убираем Panel::before, так как не несет за собой никакой функции. Описывая фон, её полностью перекрывает Panel__in, в том числе не видно разницы в режиме нескольких колонок, со скролом на странице, с разным значением aria-inset.

By setting it using token
Copy link
Contributor

github-actions bot commented Mar 22, 2024

size-limit report 📦

Path Size
JS 356.88 KB (0%)
JS (gzip) 108.94 KB (0%)
JS (brotli) 90.06 KB (0%)
JS import Div (tree shaking) 1.43 KB (0%)
CSS 261.59 KB (-0.09% 🔽)
CSS (gzip) 34.34 KB (-0.08% 🔽)
CSS (brotli) 27.82 KB (-0.17% 🔽)

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.

@mendrew mendrew added the patch Автоматизация: PR продублируется в ветку последнего минорного релиза для выпуска патча label Mar 22, 2024
@mendrew mendrew marked this pull request as ready for review March 22, 2024 11:27
@mendrew mendrew requested a review from a team as a code owner March 22, 2024 11:27
Copy link
Contributor

e2e tests

Playwright Report

Copy link
Contributor

github-actions bot commented Mar 22, 2024

👀 Docs deployed

Commit 9888370

Copy link

codecov bot commented Mar 22, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 82.38%. Comparing base (7289418) to head (9888370).
Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #6736   +/-   ##
=======================================
  Coverage   82.38%   82.38%           
=======================================
  Files         336      336           
  Lines       10359    10359           
  Branches     3469     3469           
=======================================
  Hits         8534     8534           
  Misses       1825     1825           
Flag Coverage Δ
unittests 82.38% <ø> (ø)

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.

@mendrew mendrew merged commit 780e76d into master Mar 25, 2024
49 checks passed
@mendrew mendrew deleted the mendrew/feat/6327/Panel/allow-to-use-gradient-for-background branch March 25, 2024 12:03
@vkcom-publisher
Copy link
Contributor

❌ Patch

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

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

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

  1. Создайте новую ветку от 6.0-stable и примените изменения используя cherry-pick
git stash # опционально
git fetch origin 6.0-stable
git checkout -b patch/pr6736 origin/6.0-stable

git cherry-pick --no-commit 780e76d069e35d53ec08ea986ac70b9eb4f2b5d5
git checkout HEAD **/__image_snapshots__/*.png
git diff --quiet HEAD || git commit --no-verify --no-edit
  1. Исправьте конфликты, следуя инструкциям из терминала
  2. Отправьте ветку на GitHub и создайте новый PR с веткой 6.0-stable (установка лейбла не требуется!)
git push --set-upstream origin patch/pr6736
gh pr create --base 6.0-stable --title "patch: pr6736" --body "- patch #6736"

mendrew added a commit that referenced this pull request Mar 26, 2024
- Чтобы была возможность применить градиент к фону `Panel` через токен используем свойство `background` вместо `background-color`.
- Убираем `Panel::before`, так как не несет за собой никакой функции. Описывая фон, её полностью перекрывает `Panel__in`, в том числе не видно разницы в режиме нескольких колонок, со скролом на странице, с разным значением `aria-inset`.
@mendrew mendrew mentioned this pull request Mar 26, 2024
mendrew added a commit that referenced this pull request Mar 26, 2024
- Чтобы была возможность применить градиент к фону `Panel` через токен используем свойство `background` вместо `background-color`.
- Убираем `Panel::before`, так как не несет за собой никакой функции. Описывая фон, её полностью перекрывает `Panel__in`, в том числе не видно разницы в режиме нескольких колонок, со скролом на странице, с разным значением `aria-inset`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
patch Автоматизация: PR продублируется в ветку последнего минорного релиза для выпуска патча
Projects
None yet
4 participants