Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix TouchableNativeFeedback regression #3294

Merged
merged 4 commits into from
Dec 19, 2024

Conversation

latekvo
Copy link
Contributor

@latekvo latekvo commented Dec 17, 2024

Description

#3260 introduced a regression to the Gesture Handlers's TouchableNativeFeedback component.
The component used to be a direct reference to the TouchableNativeFeedback, which has a couple of static methods.
When it was replaced with a function component, the static methods got removed, leaving only the render function of the TouchableNativeFeedback.

Test plan

See how the type errors in the example/src/release_tests/touchables/index.tsx are resolved.

console.warn(
tagMessage(
'TouchableOpacity component will be removed in the future version of Gesture Handler.'
'TouchableNativeFeedback component will be removed in the future version of Gesture Handler.'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
'TouchableNativeFeedback component will be removed in the future version of Gesture Handler.'

Let's remove this console.log as in other Touchables

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done in 13b32eb

@m-bert
Copy link
Contributor

m-bert commented Dec 17, 2024

This is what I meant when I asked if deprecation note works:

image

@latekvo
Copy link
Contributor Author

latekvo commented Dec 18, 2024

This is what I meant when I asked if deprecation note works:

fixed in 1e8e07e

@latekvo latekvo merged commit 18c39b2 into next Dec 19, 2024
1 check passed
@latekvo latekvo deleted the @latekvo/fix-deprecated-touchable-regression branch December 19, 2024 11:10
m-bert pushed a commit that referenced this pull request Dec 20, 2024
## Description

#3260 introduced a regression to the `Gesture Handlers`'s
`TouchableNativeFeedback` component.
The component used to be a direct reference to the
`TouchableNativeFeedback`, which has a [couple of static
methods](https://reactnative.dev/docs/touchablenativefeedback#methods).
When it was replaced with a function component, the static methods got
removed, leaving only the render function of the
`TouchableNativeFeedback`.

## Test plan

See how the type errors in the
`example/src/release_tests/touchables/index.tsx` are resolved.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants