some refacor about scrolableTabs

soliury · soliury · commit 7ffd3384cb41 · 2016-06-28T01:42:21.000+08:00
diff --git a/src/components/ScrollableTabs.js b/src/components/ScrollableTabs.js
@@ -99,7 +99,12 @@ class ScrollableTabs extends Component {
 
 
 	_onScroll(e) {
-		if (!this._scrolling) return;
+		const {contentSize={}} = e.nativeEvent;
+
+		// 一下一行代码为解决一个奇葩的bug
+		if (contentSize.height === 0 && contentSize.width === 0) return;
+
+
 		const {x} = e.nativeEvent.contentOffset;
 		this._scrolling = true;
 		this._animateScroll(x);
diff --git a/src/layouts/Home.js b/src/layouts/Home.js
@@ -23,9 +23,7 @@ class Home extends Component {
 		const {actions, topic, ui} = this.props;
 		const tab = Tabs.tabs[page];
 		if (topic[tab] && ui[tab] && !ui[tab].flag) {
-			setTimeout(()=> {
-				!isScrolling() && actions.updateTopicsByTab(tab);
-			}, 16);
+			actions.updateTopicsByTab(tab);
 		}
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -23,9 +23,7 @@ class Home extends Component {`
`23`	`23`	`const {actions, topic, ui} = this.props;`
`24`	`24`	`const tab = Tabs.tabs[page];`
`25`	`25`	`if (topic[tab] && ui[tab] && !ui[tab].flag) {`
`26`		`- setTimeout(()=> {`
`27`		`- !isScrolling() && actions.updateTopicsByTab(tab);`
`28`		`- }, 16);`
	`26`	`+ actions.updateTopicsByTab(tab);`
`29`	`27`	`}`
`30`	`28`	`}`
`31`	`29`