Skip to content

feat(PanelSpinner): tokenized#2614

Merged
SevereCloud merged 2 commits intomasterfrom
SevereCloud/issue2613
May 30, 2022
Merged

feat(PanelSpinner): tokenized#2614
SevereCloud merged 2 commits intomasterfrom
SevereCloud/issue2613

Conversation

@SevereCloud
Copy link
Copy Markdown
Contributor

@SevereCloud SevereCloud commented May 26, 2022

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

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

@SevereCloud SevereCloud requested a review from a team as a code owner May 26, 2022 08:46
@SevereCloud SevereCloud mentioned this pull request May 26, 2022
Closed
89 tasks
@codesandbox-ci
Copy link
Copy Markdown

codesandbox-ci bot commented May 26, 2022 •
edited
Loading

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 13a286f:

Sandbox Source
VKUI - default example Configuration

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented May 26, 2022 •
edited
Loading

size-limit report 📦

Path Size
JS 77.15 KB (+0.02% 🔺)
JS (brotli) 64.93 KB (+0.01% 🔺)
JS, unstable 29.83 KB (0%)
CSS 37.85 KB (0%)
CSS (brotli) 30.7 KB (0%)
CSS, unstable 942 B (0%)

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented May 26, 2022

👀 Styleguide deployed

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

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented May 26, 2022 •
edited
Loading

Code coverage

lines3761 / 480578.27%
statements3832 / 489778.25%
functions847 / 105480.36%
branches3126 / 443470.50%
branchesTrue0 / 0100.00%

Generated by 🚫 dangerJS against 13a286f

inomdzhon
inomdzhon reviewed May 26, 2022
View reviewed changes
@SevereCloud SevereCloud force-pushed the SevereCloud/issue2613 branch from f0fbb0e to 13a286f Compare May 26, 2022 16:44
@SevereCloud SevereCloud merged commit 5916d19 into master May 30, 2022
@SevereCloud SevereCloud deleted the SevereCloud/issue2613 branch May 30, 2022 10:00
@SevereCloud SevereCloud mentioned this pull request Jun 3, 2022
Merged
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@inomdzhon inomdzhon inomdzhon left review comments

+1 more reviewer

@stoope stoope stoope approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

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

3 participants

@SevereCloud @inomdzhon @stoope