-
Notifications
You must be signed in to change notification settings - Fork 183
add topRightView to TopBar for customizing right button on TopBar and zooming photo #20
Conversation
| /> | ||
| { | ||
| transformable ? ( | ||
| <TransformableImage |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
used this lib "react-native-transformable-image", but this has a few of conflict on pan gesture when scrolling, because it's implemented by scroll view too!
…s._updatePageIndex(this.props.initialIndex);
| const { dataSource, mediaList, useGallery } = this.props; | ||
| if (useGallery) { | ||
| return ( | ||
| <Gallery |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Using Gallery.js forked from ldn0x7dc/react-native-gallery which support costuming item for renderPage.
… rerender Gallery for reasons that you want to scroll to some other page using 'currentIndex'
|
Hi @ksti, I did tried to use it in my project, but I see the following error. Could I assume that your PR is compatible with the original example? https://github.com/7kfpun/PhotosReactNative/blob/master/app/photo-browser.js#L97-L108 |
|
@7kfpun thanks for letting me know this. Now you can 'npm update' for the latest commit. you can have a look at the usage:https://github.com/ksti/react-native-photo-browser |
|
Ok! it works, on Android and iOS. Thanks @ksti ! |
|
@ksti I would like to try your PR. but whenever I tap one of the examples in the list I get Can you help? Haven't found any additional dependencies in your example's package.json EDIT: Never mind… it has to do with the progress indicator. Maybe you can add a hint in your docs. Also in your example the urls need to be https for iOS or you get an unsecure connection error on current iOS. Impressive work btw 👍 EDIT EDIT: Noticed some sort of bug. When you specify a thumb, then no image is shown at all (using your example project). removing the thumb and it works again. |
|
@florianbepunkt Ok, thanks, I will check it soon. |
|
@florianbepunkt fixed some bugs on rn0.39 and updated the example, wish you like it :) |
|
Eagerly awaiting this merge! :) |
|
Hey @ksti ! Thanks a lot for your time and contributions. I was busy for maintaining this repo for a while and couldn't review your PRs before. So sorry about that. I think this PR has gotten really huge due to my fault. I'm going to checkout your fork and we can discuss if we could merge your fork over this one. |
|
I am looking forward to this PR. |
|
any update on this? |
|
Bump |
|
any updates? |
|
+1 for this PR |
|
+1 for zoom photo. |
|
Is this #PR merged or not? It's almost year. |
|
The diff between the project and this PR have gone huge due to my inability to respond this PR in time. I'm sorry about that. I just created a separate issue for building zoom support: #89. I think we should use some other library for zooming, and https://github.com/ascoders/react-native-image-viewer looks like a good candidate. Contributions are greatly appreciated. |

add topRightView to TopBar for customizing right button on TopBar
onTopRight is the handler for onPress event
zooming photo used this lib "react-native-transformable-image", but this has a few of conflict on pan gesture when scrolling, because it's implemented by scroll view too! And I made some PR like react-native-view-pager and react-native-gallery for resolving these problems.