Skip to content

Commit

Permalink
Fix onMouseEnter/onMouseLeave callbacks not firing on Pressable (face…
Browse files Browse the repository at this point in the history
…book#855)

* add pull yml

* match handleOpenURLNotification event payload with iOS (facebook#755) (facebook#2)

Co-authored-by: Ryan Linton <ryanlntn@gmail.com>

* fix mouse evetns on pressable

* delete extra yml from this branch

* Add macOS tags

* reorder props to have onMouseEnter/onMouseLeave always be before onPress

Co-authored-by: pull[bot] <39814207+pull[bot]@users.noreply.github.com>
Co-authored-by: Ryan Linton <ryanlntn@gmail.com>
  • Loading branch information
3 people authored Oct 13, 2021
1 parent 0cf2b3d commit c979f2e
Show file tree
Hide file tree
Showing 3 changed files with 37 additions and 29 deletions.
6 changes: 6 additions & 0 deletions Libraries/Components/Pressable/Pressable.js
Original file line number Diff line number Diff line change
Expand Up @@ -170,6 +170,8 @@ function Pressable(props: Props, forwardedRef): React.Node {
delayLongPress,
disabled,
focusable,
onMouseEnter, // [TODO(macOS GH#774)
onMouseLeave, // ]TODO(macOS GH#774)
onLongPress,
onPress,
onPressIn,
Expand Down Expand Up @@ -208,6 +210,8 @@ function Pressable(props: Props, forwardedRef): React.Node {
android_disableSound,
delayLongPress,
delayPressIn: unstable_pressDelay,
onHoverIn: onMouseEnter, // [TODO(macOS GH#774)
onHoverOut: onMouseLeave, // ]TODO(macOS GH#774)
onLongPress,
onPress,
onPressIn(event: PressEvent): void {
Expand Down Expand Up @@ -236,6 +240,8 @@ function Pressable(props: Props, forwardedRef): React.Node {
delayLongPress,
disabled,
hitSlop,
onMouseEnter, // [TODO(macOS GH#774)
onMouseLeave, // ]TODO(macOS GH#774)
onLongPress,
onPress,
onPressIn,
Expand Down
58 changes: 29 additions & 29 deletions packages/rn-tester/Podfile.lock
Original file line number Diff line number Diff line change
Expand Up @@ -510,8 +510,8 @@ SPEC CHECKSUMS:
CocoaAsyncSocket: 694058e7c0ed05a9e217d1b3c7ded962f4180845
CocoaLibEvent: 2fab71b8bd46dd33ddb959f7928ec5909f838e3f
DoubleConversion: 0ea4559a49682230337df966e735d6cc7760108e
FBLazyVector: f3a2bccaa26e739c3ef011f4ca86c3ca5427eee6
FBReactNativeSpec: 284873b891962e396668ece89aa3290221b560f6
FBLazyVector: 3e1c4c346ec478750c07d0f1d89b424d96cac528
FBReactNativeSpec: 49208bf2f791afd2253440847299776bdcdbc96f
Flipper: be611d4b742d8c87fbae2ca5f44603a02539e365
Flipper-DoubleConversion: 38631e41ef4f9b12861c67d17cb5518d06badc41
Flipper-Folly: c12092ea368353b58e992843a990a3225d4533c3
Expand All @@ -522,35 +522,35 @@ SPEC CHECKSUMS:
glog: 0dc7efada961c0793012970b60faebbd58b0decb
OpenSSL-Universal: 8b48cc0d10c1b2923617dfe5c178aa9ed2689355
RCT-Folly: 2f2111690f1e23490285c059ca53be22fe6d6bee
RCTRequired: 892b1fc61efc4f66a92b9c6ca85e2522e051f0e2
RCTTypeSafety: 94bdbb3ebfd2fcc54174c5eb66486cbfac50fe60
React: c6d1cfc7cb81cde55e07be8c1411fcdb14161562
React-callinvoker: 577cf455f9e762be4e2b03e03fc0b267a2cff771
React-Core: d8c133ae796db1854204a1f260ef6e3837f1285c
React-CoreModules: 23672f86dd25805b0a9096692b139e26116c1980
React-cxxreact: 1f21c1a08237692657816d8092f3d1dc6f13fb41
React-jsi: e42a0922ac3286bbfc1840aebba8e41fdfcc98d7
React-jsiexecutor: 161f172db05ec4ae77be65940206ba29930355a4
React-jsinspector: 680bd24100071d26698fef4e9c75cf69db027954
React-perflogger: 3a28a552bc5c0be0e8cdcf9f2977aea59eba7380
React-RCTActionSheet: 06144b4bb931c8f217afba1cd620a682cce980fa
React-RCTAnimation: 8b89532952beb474e0d2b3fbe1bdb3e107026551
React-RCTBlob: c83099e47b7a9c1aeeb584b8696abc9f78adf3f1
React-RCTImage: 69061c1547f728421a0aa09a2f1147f59c0d873c
React-RCTLinking: 341ac25287553e51b319266245ef03c36169c671
React-RCTNetwork: 1ab7893ac9d21d03da65ac958271405def8fe43a
React-RCTPushNotification: e5353eb3f501048e032cda7f9491105cc70039a0
React-RCTSettings: fa2839bad861f0d44c223938658e9e7782074c55
React-RCTTest: ca048e75c0b296329201ea81b9f14d27ddd41823
React-RCTText: d2c121533280c59313f002e6575118b877dd40fc
React-RCTVibration: c53993ba40851fc666c350da5269556b1c1470a4
React-runtimeexecutor: 87ba843187b8bebb4111c8c54f199cd4785d2ee3
RCTRequired: 6ef1ca4d8dc14cfdd1c37f14eb4137badd10e982
RCTTypeSafety: 903e3df81eaa3809cd3593f10de14c4bdd29f500
React: c49df3a16180bd95f195f359dd2e08555147b95d
React-callinvoker: aec8865b89dbb1cd8d6cbe782badf4d7495682c9
React-Core: 27db1d573718bb6d53fef2f1dc93c1fdb0d5699d
React-CoreModules: 554ae8216e1963fcfc0ba8aa539a69ad0e0e64ab
React-cxxreact: 841ef739ed5009a9a2e034af90fd7c5157a44790
React-jsi: 4b7f4a5733a77130579aa2b2af3c7963ea5f58c0
React-jsiexecutor: 56d0bf876000abf04e757f3cbd6795d257466b6d
React-jsinspector: 6ddd209772e5b066824a030da0e81a949c8bc45b
React-perflogger: 112849bd1692d40371bcb528e6cd37f6e97da42e
React-RCTActionSheet: b2bedbbb047c24e34304e5e6332182c81fbde3c4
React-RCTAnimation: 88a7a4272ba2142484d5310fa7450d2e65e58cea
React-RCTBlob: 19acefd489b5c3493a92c1f5c0e3d9eac8d05e46
React-RCTImage: 723ec9d61d7f68e6be2e28fb7064d29902c534ac
React-RCTLinking: 2d7559d3c705da2474cd9c47351d029600a1f037
React-RCTNetwork: 36c561fb46ebe56891a401fa6559fd6eae1fcf33
React-RCTPushNotification: 993a3510ea377519306a04d69970ebc9867e25e2
React-RCTSettings: f3e5ffe73ea7831d98b2d0a119102d9ba232774c
React-RCTTest: d56fa23c7ae8c29236cfb2cd7389337c81f6455b
React-RCTText: 627606cbb59d72a924eed4257ebae197b0570757
React-RCTVibration: b212ea311c7b325cabd4d780cc337cff23d40f63
React-runtimeexecutor: 04275caf6c4b2128dcbafcfa015def1782d84760
React-TurboModuleCxx-RNW: 12172bdbaaf052406ec571465243fad4b2eb2702
React-TurboModuleCxx-WinRTPort: 32917c4b48bbe4bb20ce172c40c3cd6315354c3e
ReactCommon: f4a3480837fc8e5c73ffc38c057594b3a1ec7a43
Yoga: 9defa6fd4bf71cdbba9b1389adb9deed6255f4b1
React-TurboModuleCxx-WinRTPort: 9fbcbcbfed07bdacde7fde7f85773e705f2a9c1e
ReactCommon: 91962a733289a96984bc8c761b08ed23c0fbe5f8
Yoga: 9cd54ce33d6f399a07a78172b0c8d7b89a9fc2c5
YogaKit: f782866e155069a2cca2517aafea43200b01fd5a

PODFILE CHECKSUM: cb260f8f7765c910b68f8267cbd74709f6ae6e54

COCOAPODS: 1.10.1
COCOAPODS: 1.11.0
2 changes: 2 additions & 0 deletions packages/rn-tester/js/examples/Pressable/PressableExample.js
Original file line number Diff line number Diff line change
Expand Up @@ -99,6 +99,8 @@ function PressableFeedbackEvents() {
testID="pressable_feedback_events_button"
accessibilityLabel="pressable feedback events"
accessibilityRole="button"
onMouseEnter={() => appendEvent('mouseEnter')} // [TODO(macOS GH#774)
onMouseLeave={() => appendEvent('mouseLeave')} // ]TODO(macOS GH#774)
onPress={() => appendEvent('press')}
onPressIn={() => appendEvent('pressIn')}
onPressOut={() => appendEvent('pressOut')}
Expand Down

0 comments on commit c979f2e

Please sign in to comment.