search

githuboftigran / rn-range-slider

A native slider with range
MIT License
237 stars 133 forks source link

Doesn't work on Expo Cli #33

Closed wmortume closed 4 years ago

wmortume commented 4 years ago

Getting this error. Have anyone managed to get it to work on expo?

image

satoshisound commented 4 years ago

Same issue here.

gr0x0 commented 4 years ago

Any news?

githuboftigran commented 4 years ago

@clu7ch3r , @robertwilliamallen , @gr0x0 , won't fix this as I'm not using expo. I prefer using react native cli, build project on my machine and don't take Expo seriously.

githuboftigran commented 4 years ago

@clu7ch3r , there is already a line in known issues :)

githuboftigran commented 4 years ago

@clu7ch3r I mean Known issues section of readme doc image

githuboftigran commented 4 years ago

@clu7ch3r, that's why you should always read the documentation and scroll to known issues section before using a library :) Hope this case will help you to be more attentive to readme files and documentations in future.

marcoshevaristo commented 3 years ago

@githuboftigran I've read the documentation and there is no mentioning of the event "onTouchEnd" for example, but it exists and it works perfectly (tried 3 hole days making this behavior myself before knowing this was already a thing). So what is the point of reading a documentation that is not reliable?

githuboftigran commented 3 years ago

@doomevaristo , onTouchEnd doesn't work properly. The library passes all additional props to root view, so it's a View prop. Slider uses pan handlers and other touch handlers shouldn't be used.

Don't know why do you need it and why it works for you, but passing onTouchEnd breaks the component on emulator.