Skip to content

feat(Slider): tokenized#2683

Merged
SevereCloud merged 2 commits intomasterfrom
SevereCloud/issue2575
Jun 14, 2022
Merged

feat(Slider): tokenized#2683
SevereCloud merged 2 commits intomasterfrom
SevereCloud/issue2575

Conversation

@SevereCloud
Copy link
Copy Markdown
Contributor

Чеклист перевода компонента на vkui-tokens

  • Компонент добавлен в src/tokenized/index.tssrc/index.ts он так же должен быть)
  • Имя компонента добавлено в массив из styleguide/tokenized.js
  • Если в стилях встречаются токены из Appearance, то их нужно не удалять, а дополнять фоллбэком на соответствующий токен из vkui-tokens (пример такого PR #2647)
  • Исключаем проверки типа platform === ANDROID (пример такого PR #2653)
  • В стилях компонента не осталось платформенных селекторов
  • В tsx компонента не осталось логики, которая зависит от платформы

@SevereCloud SevereCloud requested a review from a team as a code owner June 8, 2022 10:22
@codesandbox-ci
Copy link
Copy Markdown

codesandbox-ci bot commented Jun 8, 2022

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 d2b3f74:

Sandbox Source
VKUI - default example Configuration

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jun 8, 2022

size-limit report 📦

Path Size
JS 257.92 KB (-0.01% 🔽)
JS (gzip) 76.37 KB (+0.03% 🔺)
JS (brotli) 64.4 KB (-0.02% 🔽)
JS, unstable 29.56 KB (0%)
CSS 271.44 KB (-0.11% 🔽)
CSS (gzip) 38.1 KB (-0.12% 🔽)
CSS (brotli) 30.79 KB (-0.29% 🔽)
CSS, unstable 942 B (0%)

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jun 8, 2022

👀 Styleguide deployed

See the styleguide for this PR at https://vkcom.github.io/VKUI/pull/2683/

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jun 8, 2022

Warnings
⚠️

3 changed screenshots found — review & update them via "Update Screenshots" action before merging.

Changed screenshots

slider-android-light-1
slider-ios-light-1
slider-vkcom-light-1

Code coverage

lines3800 / 484478.44%
statements3869 / 493678.38%
functions842 / 105379.96%
branches3232 / 454471.12%
branchesTrue0 / 0100.00%

Generated by 🚫 dangerJS against 49f4b55

@SevereCloud SevereCloud marked this pull request as draft June 8, 2022 11:25
@SevereCloud SevereCloud force-pushed the SevereCloud/issue2575 branch 2 times, most recently from d0c56d4 to 7c743de Compare June 10, 2022 10:31
@SevereCloud SevereCloud marked this pull request as ready for review June 10, 2022 10:42
@SevereCloud
Copy link
Copy Markdown
Contributor Author

Дизайн ревью ✅ (by @Fliqle)

stoope
stoope previously approved these changes Jun 10, 2022
@SevereCloud SevereCloud force-pushed the SevereCloud/issue2575 branch from 01e00b6 to 49f4b55 Compare June 14, 2022 07:59
@SevereCloud SevereCloud merged commit fa9d663 into master Jun 14, 2022
@SevereCloud SevereCloud deleted the SevereCloud/issue2575 branch June 14, 2022 11:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature] Токенизировать RangeSlider [Feature] Токенизировать Slider

3 participants