Skip to content

Conversation

@Wmeng98
Copy link

@Wmeng98 Wmeng98 commented Jan 12, 2020

Is there a need to keep this forked module up to date with the latest commits on ptomasroos original repo?

janicduplessis and others added 30 commits April 22, 2018 19:45
Trailing commas are not allowed after the rest element, this is not enforced by babel 6 but is by babel 7.
Fix invalid object rest syntax
Removed trailing comma, which is a breaking change for RN 0.56
use __getValue() instead of ._value
Update ScrollableTabsExample.js
* remove deprecated ViewPager

* Update index.js

Co-Authored-By: Mustafa Buyukcelebi <[email protected]>
* add @react-native-community/viewpager in package.json

* modify ViewPagerAndroid to refer to @react-native-community/viewpager

* change version of @react-native-community/viewpager to ^2.0.1
* adds passing onScroll to ScrollView in ScrollableTabBar

* adds throttle to onScroll events of the underlying ScrollView
`onPageSelected={this._updateSelectedPage}` ViewPager prop causes onTabChange to be called twice when change tabs without any gesture actions
this will fix duplicated tab updates on android

#### when user clicked tab buttons
`goToPage -> updateSceneKeys(onTabChange) -> Screen Update -> onPageSelected={_updateSelectedPage} -> updateSceneKeys(onTabChange) -> Screen Update`

#### when user swiped tab views
`Swipe -> onPageSelected={_updateSelectedPage} -> updateSceneKeys(onTabChange) -> Screen Update`

#### when user clicked tab buttons with this patch
`goToPage -> set tabWillChangeWithoutGesture -> updateSceneKeys(onTabChange) -> Screen Update -> onPageSelected={_updateSelectedPage} -> ignore updateSceneKeys(onTabChange) -> remove tabWillChangeWithoutGesture`
…ineStyle] (#1055)

tabBarUnderlineStyle was not working on default tab bar because it was excepting tabUnderlineStyle prop
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.