-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Avoid player jumping on devices with a cutout and fix two taps required for interacting #9785
Avoid player jumping on devices with a cutout and fix two taps required for interacting #9785
Conversation
d0191aa
to
383a322
Compare
I was about to mention this problem. If you're in landscape full screen, and you try to swipe up on the Full Screen button to scroll down the video details page, the player just starts jumping and preventing the gesture from working. Hopefully this fixes it. |
@SameenAhnaf could you check if this also fixes your "requires two taps" problem? |
@Stypox Nope, this PR doesn't fix that bug |
@Isira-Seneviratne then I think it's best to just revert #8894. It required to fix many bugs before merging, and even after merging new bugs keep arising. I don't think it's worth it to use |
Actually, I found out all flags were deprecated, so let's try a bit more before reverting. I pushed a commit that properly sets window flags only when needed, and also does not use |
@Stypox I simplified one of the methods you added. |
Kudos, SonarCloud Quality Gate passed! |
What is it?
Description of the changes in your PR
Fixes the following issue(s)
APK testing
The APK can be found by going to the "Checks" tab below the title. On the left pane, click on "CI", scroll down to "artifacts" and click "app" to download the zip file which contains the debug APK of this PR.
Due diligence