react-native-slider icon indicating copy to clipboard operation
react-native-slider copied to clipboard

Use `ImageURISource` for thumb image instead of `ImageSource` to fix image not loading

Open BartoszKlonowski opened this issue 7 months ago • 1 comments

This pull request fixes #711

It migrates from ImageSource type from RN to ImageURISource. Those are both from RN and have common props however ImageSource is no longer supported by the v0.79 app (and beyond).

This fix includes both iOS and Android. Also, all dev deps were updated to make sure we have the recent types used within the package when consumed by the app.

Note:: Removing tests from workflow is temporary due to versions update and tests now failing (some features of React test were used previously).

BartoszKlonowski avatar Jun 05 '25 12:06 BartoszKlonowski

Looking forward to merging into main line release soon, thanks

Qdafengzi avatar Jun 18 '25 09:06 Qdafengzi

@BartoszKlonowski I wanted to check to see if you have any updates as to when this might be merged and released? We're unable to use a custom thumbImage in newer versions of React Native (e.g. 0.80.2).

brianlenz avatar Sep 05 '25 17:09 brianlenz

@BartoszKlonowski FWIW, I forked the repo and merged the changes here into 5.0.1. It fixes the issue with the iOS thumb image not working 🙏

brianlenz avatar Sep 09 '25 18:09 brianlenz

@brianlenz @Qdafengzi Thank you for the interest and checking the solution! It's been a while since I had a possibility to sit down to it, but in the meantime, together with @okwasniewski (thanks!) we figured out that what is implemented in this PR is rather a workaround and we should rather stick to ImageSource. Considering input provided in #734 by @pingkai there is an option to figure out a better solution soon. I'm hoping for your patience while we'll be pushing the topic forward, and in the meantime please reach out to a workaround suggested in #734. I will keep this thread active as soon as there will be more findings or ETAs. Thank you!

BartoszKlonowski avatar Sep 18 '25 18:09 BartoszKlonowski

mark

lapidcj avatar Sep 21 '25 14:09 lapidcj

Closing as superseded by #736

BartoszKlonowski avatar Oct 24 '25 11:10 BartoszKlonowski