kylefang/react-native-web
1
0
Fork 0
mirror of https://github.com/zhigang1992/react-native-web.git synced 2026-04-30 05:14:53 +08:00
Code Issues Packages Projects Releases Wiki Activity

[fix] View event handling

Browse Source 
Fix #238
This commit is contained in:
Nicolas Gallagher
2016-10-27 21:00:17 -07:00
parent 0af6dc00fc
commit 14072c7471
2 changed files with 4 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
examples/components/ScrollView/ScrollViewExample.js
View File

@@ -1,6 +1,6 @@
import React from 'react'; import React from 'react';
import { storiesOf, action } from '@kadira/storybook'; import { storiesOf, action } from '@kadira/storybook';
import { ScrollView, StyleSheet, Text, View } from 'react-native' import { ScrollView, StyleSheet, Text, TouchableHighlight, View } from 'react-native'
storiesOf('component: ScrollView', module) storiesOf('component: ScrollView', module)
.add('vertical', () => ( .add('vertical', () => (
@@ -13,7 +13,7 @@ storiesOf('component: ScrollView', module)
> >
{Array.from({ length: 50 }).map((item, i) => ( {Array.from({ length: 50 }).map((item, i) => (
<View key={i} style={styles.box}> <View key={i} style={styles.box}>
<Text>{i}</Text> <TouchableHighlight onPress={() => {}}><Text>{i}</Text></TouchableHighlight>
</View> </View>
))} ))}
</ScrollView> </ScrollView>
@@ -39,7 +39,6 @@ storiesOf('component: ScrollView', module)
const styles = StyleSheet.create({ const styles = StyleSheet.create({
box: { box: {
alignItems: 'center',
flexGrow: 1, flexGrow: 1,
justifyContent: 'center', justifyContent: 'center',
borderWidth: 1 borderWidth: 1

4
src/components/View/index.js
View File

@@ -124,10 +124,10 @@ class View extends Component {
_normalizeEventForHandler(handler, handlerName) { _normalizeEventForHandler(handler, handlerName) {
// Browsers fire mouse events after touch events. This causes the // Browsers fire mouse events after touch events. This causes the
// ResponderEvents and their handlers to fire twice for Touchables. // 'onResponderRelease' handler to be called twice for Touchables.
// Auto-fix this issue by calling 'preventDefault' to cancel the mouse // Auto-fix this issue by calling 'preventDefault' to cancel the mouse
// events. // events.
const shouldCancelEvent = handlerName.indexOf('onResponder') === 0; const shouldCancelEvent = handlerName === 'onResponderRelease';
return (e) => { return (e) => {
e.nativeEvent = normalizeNativeEvent(e.nativeEvent); e.nativeEvent = normalizeNativeEvent(e.nativeEvent);