nebius/SWE-rebench-V2-PRs huggingface dataset:
Repository: callstack/react-native-paper
Pull request #1295 (2019-08-21 12:38:55)

fix: change chip close icon to material community icon
Current situation with v3 is the left but it needs to be the right icon like in v2 ;)

I can't fix the test because the only thing is see is a square in the snapshots.

![Schermafbeelding 2019-08-21 om 14 39 33](https://user-images.githubusercontent.com/6492229/63432476-86709200-c421-11e9-8423-60d4fdaa9150.png)


callstack-bot: Hey @RichardLindhout, thank you for your pull request 🤗. The documentation from this branch can be viewed [here](https://12400-71323749-gh.circle-artifacts.com/0/docs/index.html).
jaysbytes: Hi @RichardLindhout. You can fix test using `yarn test -u`. It should update snapshots and then commit everything and push.
RichardLindhout: Mind=blown been doing all the snapshots updates myself last time xD
RichardLindhout: @jaysbytes Done, had some difficulties with squashing. Did it for the first time, but I think it should be fine now!
jaysbytes: @RichardLindhout 
> Mind=blown been doing all the snapshots updates myself last time xD
It happens :D But look, now you have a piece of new knowledge and also contribution.
Also, you don't have to worry about squashing, we will squash your commit anyway while merging a PR


Patch:
diff --git a/src/components/Chip.tsx b/src/components/Chip.tsx
index ed962039d..6fd5e953c 100644
--- a/src/components/Chip.tsx
+++ b/src/components/Chip.tsx
@@ -302,7 +302,7 @@ class Chip extends React.Component<Props, State> {
                 accessibilityComponentType="button"
               >
                 <View style={styles.icon}>
-                  <Icon source="cancel" size={16} color={iconColor} />
+                  <Icon source="close-circle" size={16} color={iconColor} />
                 </View>
               </TouchableWithoutFeedback>
             ) : null}


Test patch:
diff --git a/src/components/__tests__/__snapshots__/Chip.test.js.snap b/src/components/__tests__/__snapshots__/Chip.test.js.snap
index 0d1b156a4..594883973 100644
--- a/src/components/__tests__/__snapshots__/Chip.test.js.snap
+++ b/src/components/__tests__/__snapshots__/Chip.test.js.snap
@@ -169,7 +169,7 @@ exports[`renders chip with close button 1`] = `
             ]
           }
         >
-          
+          
         </Text>
       </View>
     </View>
Repository: callstack/react-native-paper
Pull request #1422 (2019-10-25 12:48:15)

fix: allow touch events to go through in PortalHost
### Motivation

I was running into this issue https://github.com/wix/react-native-navigation/issues/5579 earlier.
Turns out that wrapping my screens in a ThemeProvider wasn't allowing my touches to go thru.
This should fix the issue.



callstack-bot: Hey @forkandfix, thank you for your pull request 🤗. The documentation from this branch can be viewed [here](https://13212-71323749-gh.circle-artifacts.com/0/docs/index.html).
forkandfix: lol sorry by mistake
Trancever: @forkandfix Please make sure to update snapshot tests and fix lint issues.
forkandfix: @Trancever done

Patch:
diff --git a/src/components/Menu/MenuItem.tsx b/src/components/Menu/MenuItem.tsx
index d80d915bb..d0088316e 100644
--- a/src/components/Menu/MenuItem.tsx
+++ b/src/components/Menu/MenuItem.tsx
@@ -1,6 +1,6 @@
 import color from 'color';
 import * as React from 'react';
-import { View, StyleSheet } from 'react-native';
+import { View, StyleSheet, ViewStyle, StyleProp } from 'react-native';
 import Icon, { IconSource } from '../Icon';
 import TouchableRipple from '../TouchableRipple';
 import Text from '../Typography/Text';
@@ -29,7 +29,7 @@ type Props = {
    * @optional
    */
   theme: Theme;
-  style?: any;
+  style?: StyleProp<ViewStyle>;
 };
 
 /**
diff --git a/src/components/Portal/PortalHost.tsx b/src/components/Portal/PortalHost.tsx
index 4a8dc81c5..7d03069b4 100644
--- a/src/components/Portal/PortalHost.tsx
+++ b/src/components/Portal/PortalHost.tsx
@@ -125,7 +125,11 @@ export default class PortalHost extends React.Component<Props> {
         }}
       >
         {/* Need collapsable=false here to clip the elevations, otherwise they appear above Portal components */}
-        <View style={styles.container} collapsable={false}>
+        <View
+          style={styles.container}
+          collapsable={false}
+          pointerEvents="box-none"
+        >
           {this.props.children}
         </View>
         <PortalManager ref={this.setManager} />


Test patch:
diff --git a/src/components/__tests__/__snapshots__/Portal.test.js.snap b/src/components/__tests__/__snapshots__/Portal.test.js.snap
index ab38d9acc..da6b2eb94 100644
--- a/src/components/__tests__/__snapshots__/Portal.test.js.snap
+++ b/src/components/__tests__/__snapshots__/Portal.test.js.snap
@@ -4,6 +4,7 @@ exports[`renders portal with siblings 1`] = `
 Array [
   <View
     collapsable={false}
+    pointerEvents="box-none"
     style={
       Object {
         "flex": 1,