Skip to content

refactor: PullToRefresh to FC#2660

Merged
inomdzhon merged 1 commit intomasterfrom
feat/2458/pullToRefresh_to_FC
Jun 6, 2022
Merged

refactor: PullToRefresh to FC#2660
inomdzhon merged 1 commit intomasterfrom
feat/2458/pullToRefresh_to_FC

Conversation

@BlackySoul
Copy link
Copy Markdown
Contributor

@BlackySoul BlackySoul commented Jun 3, 2022

@BlackySoul BlackySoul requested a review from a team as a code owner June 3, 2022 11:28
@BlackySoul BlackySoul linked an issue Jun 3, 2022 that may be closed by this pull request
@codesandbox-ci
Copy link
Copy Markdown

codesandbox-ci bot commented Jun 3, 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 f42e42a:

Sandbox Source
VKUI - default example Configuration

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jun 3, 2022

size-limit report 📦

Path Size
JS 257.49 KB (-0.61% 🔽)
JS (gzip) 76.63 KB (-0.36% 🔽)
JS (brotli) 64.51 KB (-0.37% 🔽)
JS, unstable 29.57 KB (0%)
CSS 266.61 KB (0%)
CSS (gzip) 37.69 KB (0%)
CSS (brotli) 30.6 KB (0%)
CSS, unstable 942 B (0%)

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jun 3, 2022

👀 Styleguide deployed

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

@BlackySoul BlackySoul force-pushed the feat/2458/pullToRefresh_to_FC branch from 6b4ca5b to 8bc5686 Compare June 3, 2022 11:37
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jun 3, 2022

Code coverage

lines3784 / 482378.45%
statements3854 / 491678.39%
functions842 / 105379.96%
branches3155 / 445470.83%
branchesTrue0 / 0100.00%

Generated by 🚫 dangerJS against f42e42a

inomdzhon
inomdzhon previously approved these changes Jun 3, 2022
Copy link
Copy Markdown
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.

💖

onMove={onTouchMove}
onEnd={onTouchEnd}
vkuiClass={classNames(
getClassName("PullToRefresh", platform),
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Давай заменим на platform === IOS? Вижу знания о android и vkcom уже не нужны, соответственно getClassName не нужон

Copy link
Copy Markdown
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.

Ой, случайно апрув нажал 😅

@inomdzhon inomdzhon merged commit 7fa971d into master Jun 6, 2022
@inomdzhon inomdzhon deleted the feat/2458/pullToRefresh_to_FC branch June 6, 2022 09:57
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.

Переход PullToRefresh на FC

2 participants