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

feat: added onLongPress to RadioButton.Item and Checkbox.Item #4215

Merged
merged 2 commits into from
Jan 8, 2024
Merged

feat: added onLongPress to RadioButton.Item and Checkbox.Item #4215

merged 2 commits into from
Jan 8, 2024

Conversation

ondrejnedoma
Copy link
Contributor

Motivation

RadioButton.Item currently does not pass the "onLongPress" prop to the TouchableRipple component. This would be useful to implement to make RadioButton.Item elements be able to handle a secondary action.

Related issue

#4214

Test plan

Create a RadioButton.Item and pass it an onLongPress prop with a function that logs something into the console.

@callstack-bot
Copy link

callstack-bot commented Dec 5, 2023

Hey @ondrejnedoma, thank you for your pull request 🤗. The documentation from this branch can be viewed here.

Copy link
Contributor

@kaushik1094 kaushik1094 left a comment

Choose a reason for hiding this comment

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

Thanks for making necessary changes.

@lukewalczak lukewalczak added this to the 5.12.0 milestone Dec 11, 2023
@lukewalczak lukewalczak linked an issue Dec 11, 2023 that may be closed by this pull request
@lukewalczak
Copy link
Member

Hey @ondrejnedoma, please rename the commit message to follow the library convention

@ondrejnedoma
Copy link
Contributor Author

Fixed the name, also took me a while to figure out how fork syncing works (or does not work) so sorry for the spam :D

@lukewalczak lukewalczak changed the title Added onLongPress to RadioButton.Item feat: added onLongPress to RadioButton.Item Jan 8, 2024
@lukewalczak lukewalczak changed the title feat: added onLongPress to RadioButton.Item feat: added onLongPress to RadioButton.Item and Checkbox.Item Jan 8, 2024
@lukewalczak lukewalczak merged commit 4d57fe2 into callstack:main Jan 8, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

RadioButton.Item does not take onLongPress
4 participants