Skip to content
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

Player: Swipe to minimize #456

Closed

Conversation

juniorrantila
Copy link
Contributor

This patch adjusts the distance you have to swipe to minimize the
player from 50% of the view height to 10%, making it significantly
easier minimize the player.

This patch also makes it so that you can swipe down from the middle
of the screen to minimize the player. If the volume/brightness gestures
are disabled, you're able to swipe down from anywhere on the screen.

Before

before

After

after

This decreases the distance you have to swipe to minimize both
PlayerViewController and VideoPlayerViewController
This patch makes it so that you can swipe down from the middle of
the screen to minimize the player. If the volume/brightness gesture
are disabled, you're able to swipe down from anywhere on screen.
@fkuehne
Copy link
Contributor

fkuehne commented Aug 8, 2023

Thanks a lot for your pull request, which is a very nice improvement! We cannot merge it here, so I created a merge request on our own gitlab instance here: https://code.videolan.org/videolan/vlc-ios/-/merge_requests/1092

You are very welcome to continue contributing to our project!

@fkuehne fkuehne closed this Aug 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants