fix: message longPress not working. (#1018)

zulip · Aug 12, 2017 · 07fe489 · 07fe489
1 parent 7fbe5bd
commit 07fe489
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 10 deletions.
diff --git a/src/message/MessageBrief.js b/src/message/MessageBrief.js
@@ -44,11 +44,13 @@ export default class MessageBrief extends PureComponent {
     return (
       <View style={styles.message}>
         <View style={styles.messageContentWrapper}>
-          <TouchableWithoutFeedback onLongPress={onLongPress}>
-            <ScrollView style={styles.childrenWrapper}>
-              {children}
-            </ScrollView>
-          </TouchableWithoutFeedback>
+          <ScrollView style={styles.childrenWrapper}>
+            <TouchableWithoutFeedback onLongPress={onLongPress}>
+              <View>
+                {children}
+              </View>
+            </TouchableWithoutFeedback>
+          </ScrollView>
         </View>
         <MessageTags timestamp={message.last_edit_timestamp} starred={starred} />
         <ReactionList messageId={message.id} reactions={message.reactions} ownEmail={ownEmail} />

diff --git a/src/message/MessageFull.js b/src/message/MessageFull.js
@@ -63,11 +63,13 @@ class MessageFull extends PureComponent {
             twentyFourHourTime={twentyFourHourTime}
           />
           <View style={styles.contentWrapper}>
-            <TouchableWithoutFeedback onLongPress={onLongPress}>
-              <ScrollView style={styles.inner}>
-                {children}
-              </ScrollView>
-            </TouchableWithoutFeedback>
+            <ScrollView style={styles.inner}>
+              <TouchableWithoutFeedback onLongPress={onLongPress}>
+                <View>
+                  {children}
+                </View>
+              </TouchableWithoutFeedback>
+            </ScrollView>
           </View>
           <MessageTags timestamp={message.last_edit_timestamp} starred={starred} />
           <ReactionList messageId={message.id} reactions={message.reactions} ownEmail={ownEmail} />