-
Notifications
You must be signed in to change notification settings - Fork 552
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
Small improvements #1
Comments
Those are really good suggestions, thanks. |
Thanks to #3 and recent commits, i think all of those suggestions are covered by now. |
FIX: transitionOnUserGestures value not used
FIX: transitionOnUserGestures value not used
FIX: transitionOnUserGestures value not used
FIX: transitionOnUserGestures value not used
I still see the back background. I'm using the PUB version. |
Hmm I just noticed you added a backgroundDecoration prop. |
Probably there are already tons of people using PhotoView by now. I will avoid major layout chages like this default swap. It is pretty simple to customize it. |
Makes sense. It's good that you think this way. |
First I want to thank you for this widget, it will be very useful if you can add:
The text was updated successfully, but these errors were encountered: