Do not compare unnecessary prop values in modifiers (#543)

Author: emilisb

src/commons/baseComponent.js

@@ -83,7 +83,8 @@ export default function baseComponent(usePure) {
     }
 
     updateModifiers(currentProps, nextProps) {
-      const allKeys = _.union([..._.keys(currentProps), ..._.keys(nextProps)]);
+      const ignoredKeys = ['children', 'forwardedRef', 'style', 'testID'];
+      const allKeys = _.union([..._.keys(currentProps), ..._.keys(nextProps)]).filter((key) => !ignoredKeys.includes(key));
       const changedKeys = _.filter(allKeys, key => !_.isEqual(currentProps[key], nextProps[key]));
 
       const options = {};

src/commons/modifiers.js

@@ -254,7 +254,8 @@ props = this.props,) {
 }
 
 export function getAlteredModifiersOptions(currentProps, nextProps) {
-  const allKeys = _.union([..._.keys(currentProps), ..._.keys(nextProps)]);
+  const ignoredKeys = ['children', 'forwardedRef', 'style', 'testID'];
+  const allKeys = _.union([..._.keys(currentProps), ..._.keys(nextProps)]).filter((key) => !ignoredKeys.includes(key));
   const changedKeys = _.filter(allKeys, key => !_.isEqual(currentProps[key], nextProps[key]));
 
   const options = {};