zoriya/react-native-web
1
0
Fork 0
mirror of https://github.com/zoriya/react-native-web.git synced 2026-06-12 14:04:10 +00:00
Code Issues Packages Projects Releases Wiki Activity

[fix] Touchable use with react-test-renderer

Browse Source 
The object returned by 'ReactDOM.findDOMNode' when rendered by
'react-test-renderer' doesn't match the DOM API for an element. Only
attempt to bind the listener if 'addEventListener' is present on the
object.

Fix #720
This commit is contained in:
Nicolas Gallagher
2017-12-18 16:31:44 +00:00
parent 82c044ee33
commit 5f68542529
1 changed files with 18 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/components/Touchable/Touchable.js
+18 -13
View File
@@ -316,27 +316,32 @@ const LONG_PRESS_ALLOWED_MOVEMENT = 10;
* @lends Touchable.prototype
*/
const TouchableMixin = {
// HACK (part 1): basic support for touchable interactions using a keyboard
componentDidMount: function() {
this._touchableNode = findNodeHandle(this);
this._touchableBlurListener = e => {
if (this._isTouchableKeyboardActive) {
if (
this.state.touchable.touchState &&
this.state.touchable.touchState !== States.NOT_RESPONDER
) {
this.touchableHandleResponderTerminate({ nativeEvent: e });
if (this._touchableNode && this._touchableNode.addEventListener) {
this._touchableBlurListener = e => {
if (this._isTouchableKeyboardActive) {
if (
this.state.touchable.touchState &&
this.state.touchable.touchState !== States.NOT_RESPONDER
) {
this.touchableHandleResponderTerminate({ nativeEvent: e });
}
this._isTouchableKeyboardActive = false;
}
this._isTouchableKeyboardActive = false;
}
};
this._touchableNode.addEventListener('blur', this._touchableBlurListener);
};
this._touchableNode.addEventListener('blur', this._touchableBlurListener);
}
},
/**
* Clear all timeouts on unmount
*/
componentWillUnmount: function() {
this._touchableNode.removeEventListener('blur', this._touchableBlurListener);
if (this._touchableNode && this._touchableNode.addEventListener) {
this._touchableNode.removeEventListener('blur', this._touchableBlurListener);
}
this.touchableDelayTimeout && clearTimeout(this.touchableDelayTimeout);
this.longPressDelayTimeout && clearTimeout(this.longPressDelayTimeout);
this.pressOutDelayTimeout && clearTimeout(this.pressOutDelayTimeout);
@@ -791,7 +796,7 @@ const TouchableMixin = {
}
},
// HACK: basic support for touchable interactions using a keyboard (including
// HACK (part 2): basic support for touchable interactions using a keyboard (including
// delays and longPress)
touchableHandleKeyEvent: function(e: Event) {
const ENTER = 13;