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: Replacing TouableOpacity with View addresses issue 354 #355

Closed
wants to merge 3 commits into from
Closed

fix: Replacing TouableOpacity with View addresses issue 354 #355

wants to merge 3 commits into from

Conversation

deansimcox
Copy link
Contributor

@deansimcox deansimcox commented Aug 26, 2020

This PR addresses issue 354 - #354

Please advise if replacing <TouchableOpacity/> may have a negative affect.

@lfkwtz
Copy link
Collaborator

lfkwtz commented Aug 27, 2020

it will have a negative effect as it'll remove the feedback

@deansimcox
Copy link
Contributor Author

Yeah okay that makes sense.
I'll see if I can rework this PR over the next few days

@deansimcox
Copy link
Contributor Author

@lfkwtz I've not been able to figure out a way of fixing the issue and also keeping the touchable feedback.
It's almost like you would need to have the invisible on top of a , and then somehow simulate a touch event on the touchable whenever the Picker is pressed, which as far as I know, isn't possible.

@niklasnatter
Copy link

Hey, what is the status here? Is this still considered?
Unfortunately I am running into #354 and therefore I am investigating if there will be a fix or if I should look for another library.

Thanks in advance!

@lfkwtz lfkwtz mentioned this pull request Oct 14, 2020
@lfkwtz lfkwtz closed this in #377 Oct 14, 2020
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.

3 participants