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

Добавлен токен easing анимации для платформ Android и iOS #176

Merged
merged 2 commits into from
Sep 8, 2022

Conversation

Zaycevq
Copy link
Contributor

@Zaycevq Zaycevq commented Sep 2, 2022

Появился токен animationEasingPlatform, который отвечает за разную easing анимацию для Android и iOS, раньше были токены --ios-easing и --android-easing, которые находились в библиотеке VKUI, теперь их заменим на animationEasingPlatform, чтобы токен анимации жил в vkui-tokens

@BlackySoul
Copy link
Contributor

А по итогу у нас все равно будут разные токены для анимации на android/ios?

@Zaycevq
Copy link
Contributor Author

Zaycevq commented Sep 6, 2022

А по итогу у нас все равно будут разные токены для анимации на android/ios?

да, они сейчас такие и есть, токен Default для всех, а эти новые токены для разной анимации по платформам

@BlackySoul
Copy link
Contributor

А по итогу у нас все равно будут разные токены для анимации на android/ios?

да, они сейчас такие и есть, токен Default для всех, а эти новые токены для разной анимации по платформам

А нельзя один токен завести, но менять его значение в зависимости от платформы?

@Zaycevq
Copy link
Contributor Author

Zaycevq commented Sep 6, 2022

А по итогу у нас все равно будут разные токены для анимации на android/ios?

да, они сейчас такие и есть, токен Default для всех, а эти новые токены для разной анимации по платформам

А нельзя один токен завести, но менять его значение в зависимости от платформы?

так и завёл, один токен animationEasingPlatform который меняет значения в зависимости от платформы

не стал менять Default токен тк. возможно его уже применяли и он нужен в том виде который есть сейчас (если что потом удалим его)

@8coon 8coon changed the base branch from master to add-animation-easing-platform September 8, 2022 10:09
@8coon 8coon merged commit cff095b into VKCOM:add-animation-easing-platform Sep 8, 2022
This was referenced Sep 8, 2022
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.

None yet

3 participants