react-native-panorama-view icon indicating copy to clipboard operation
react-native-panorama-view copied to clipboard

Support using local images

Open mimmobordini opened this issue 5 years ago • 13 comments

is possible to use an image locally stored for using the app offline?

mimmobordini avatar Jul 30 '19 09:07 mimmobordini

Same question.

songoten28 avatar Aug 30 '19 03:08 songoten28

This is currently not supported, since we didn't have a business requirement for it. We're using the built-in image loader from React Native on iOS and a custom one for Android. So adding local images, involves writing some more native code.

You're welcome to contribute :)

rodymolenaar avatar Sep 24 '19 13:09 rodymolenaar

I've been working on some bug fixes and improvements on my own fork (https://github.com/cristianoccazinsp/react-native-panorama-view/tree/updates). The fork is still not ready/finished, and perhaps not 100% adequate for a merge, but among other things, I've added local images support.

I'm not making a PR yet since I did some readme updates for my own fork, and removed some .ts / compilation steps in order to develop and test faster (having to run a build script or setup a .ts environment was too much of a hassle), but perhaps @rodymolenaar we can come up with a good way to merge these changes once I'm done with them.

cristianoccazinsp avatar Apr 13 '20 17:04 cristianoccazinsp

Alright! Just @ping me when you're done so that I can review it :)

rodymolenaar avatar Apr 13 '20 19:04 rodymolenaar

any updates on this?

HossamSamir avatar Aug 06 '20 03:08 HossamSamir

On my fork (https://github.com/cristianoccazinsp/react-native-panorama-view/tree/updates), it is working fine with a few extra install instructions. You may fork it and use it meanwhile, or wait for @rodymolenaar to finish the merge.

cristianoccazinsp avatar Aug 06 '20 13:08 cristianoccazinsp

On my fork (https://github.com/cristianoccazinsp/react-native-panorama-view/tree/updates), it is working fine with a few extra install instructions. You may fork it and use it meanwhile, or wait for @rodymolenaar to finish the merge.

Im expecting for this function, but i dont know how to put a route to local file in your fork. Im getting a black screen in the image show. I hope you can help to use this function.

starexrex avatar Aug 06 '20 16:08 starexrex

Should be as simple as imageUrl={file://path/to/your/file}

cristianoccazinsp avatar Aug 06 '20 16:08 cristianoccazinsp

Sorry but i cant make it work

<PanoramaView style={{flex: 1}} dimensions={{ height: Dimensions.get('window').height, width: Dimensions.get('window').width, }} inputType="mono" imageUrl={'file:/' + pathName} />

pathName is something like /data/data/com.reinadoapp/filesparquebolivar.jpg

starexrex avatar Aug 06 '20 16:08 starexrex

First, you need to install the library directly from github, for example:

packages.json

"react-native-panorama-view": "github:cristianoccazinsp/react-native-panorama-view#updates",

Then, if you read the fork's install instructions, there's this:

Add the following to your project's Podfile (TODO: Remove this once CTPanoramaView is updated) pod 'CTPanoramaView', :git => 'https://github.com/cristianoccazinsp/CTPanoramaView.git', :branch => 'control-both' $ cd ios && pod install Note: You may need to delete your Podfile.lock file before running pod install

Then it should work. Keep in mind that the local file update was for files normally accessible to the app. If your file is bundled as static images, it may not work (I haven't really tested that)

cristianoccazinsp avatar Aug 06 '20 16:08 cristianoccazinsp

does this support image added inside the app ?

promit13 avatar Aug 17 '20 09:08 promit13

For images stored on the device within the app's storage (e.g., taking a photo with a camera) it does. For images stored as static resources, I haven't really tested that case. I believe it should work if the images are static resources in react native, but it probably won't if they are java/IOS static resources.

cristianoccazinsp avatar Aug 17 '20 13:08 cristianoccazinsp

@cristianoccazinsp Thanks !

promit13 avatar Aug 18 '20 10:08 promit13