Skip to content
This repository has been archived by the owner on Nov 27, 2022. It is now read-only.

feat: add animationEnabled to TabView #1388

Merged
merged 8 commits into from
Sep 30, 2022

Conversation

szymonrybczak
Copy link
Contributor

Motivation

According to this issue #1371 I added the disableChangeTabAnimation to disable animation when changing tab.

README.md Outdated Show resolved Hide resolved
src/PagerViewAdapter.tsx Outdated Show resolved Hide resolved
@szymonrybczak
Copy link
Contributor Author

@satya164 I updated my code as you said, could you please look at it again?

@szymonrybczak szymonrybczak changed the title feat: add disableChangeTabAnimation to TabView feat: add animationEnabled to TabView Sep 6, 2022
example/package.json Outdated Show resolved Hide resolved
Copy link
Owner

@satya164 satya164 left a comment

Choose a reason for hiding this comment

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

This also needs to be handled in PanResponderAdapter for Web, where it should call panX.setValue instead of performing an animation

@szymonrybczak szymonrybczak force-pushed the feat/add-disable-animation branch from 2f568d8 to 13c2037 Compare September 9, 2022 19:18
src/PanResponderAdapter.tsx Outdated Show resolved Hide resolved
src/PanResponderAdapter.tsx Outdated Show resolved Hide resolved
src/types.tsx Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@szymonrybczak szymonrybczak force-pushed the feat/add-disable-animation branch from 939cc0a to 07ee1b0 Compare September 24, 2022 12:32
@mlecoq
Copy link
Contributor

mlecoq commented Sep 28, 2022

Hi, I have made a separated PR: see #1391 because I need to be able to deactivate animation only for distant tabs (I need to keep them for neighbor tabs)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants