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: list.accordion onlongpress type #2697

Merged

Conversation

afermon
Copy link
Contributor

@afermon afermon commented May 1, 2021

Summary

Currently List.Accordion onLongPress prop has a type () => void. List.Accordion pasess this to TouchableRipple in which the type of onLongPress is (e: GestureResponderEvent) => void. When using the List.Accordion component it does passes GestureResponderEvent but since the type does not have any parameters defined the linter complains about it. List.Item uses the same type as TouchableRipple.

The GestureResponderEvent could be used to correclty locate a contextual menu over the Accordion which will be a diferent callback from onPress (expand)

Test plan

Currently this works, is just the linter since it passes the prop but the linter complains about it.

Passes type-check, lint and tests. Reviewed Docs as well.

@brunohkbx brunohkbx merged commit 75abfd0 into callstack:main Jun 22, 2021
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