-
Notifications
You must be signed in to change notification settings - Fork 7
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
Full Screen Images #4
Comments
This looks like a good solution for the time being. I mean it won't hurt functionality. |
@ryanmac8 Any chance you can put a PR in for your change? |
@viktorstrate @ryanmac8 any update on this? This would really be a huge game-changer for this app. I don't have any other major gripes. |
I opened up a PR but only the developer can merge it in, which seems unlikely due to having no response about this issue. |
Opened up a PR with a small redesign of the image/video viewing system.. It should look more like a "normal" gallery app.. #20 |
Hey,
What needs to be done to support full screen mode? I was able to remove the comments and it looks to be working fine but wanted to get your input. Also, is there a roadmap for this project? If so, I don't mind trying to help you out and get some features implemented.
.onTapGesture { // Disabled until a better solution is made mediaEnv.fullScreen = !fullscreenMode }
The text was updated successfully, but these errors were encountered: