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

Avoid player jumping on devices with a cutout and fix two taps required for interacting #9785

Conversation

Isira-Seneviratne
Copy link
Member

@Isira-Seneviratne Isira-Seneviratne commented Feb 11, 2023

What is it?

  • Bugfix (user facing)
  • Feature (user facing)
  • Codebase improvement (dev facing)
  • Meta improvement to the project (dev facing)

Description of the changes in your PR

  • Restore the code to prevent the player jumping on devices with a cutout.

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

@SameenAhnaf SameenAhnaf added feature request Issue is related to a feature in the app GUI Issue is related to the graphical user interface player Issues related to any player (main, popup and background) labels Feb 11, 2023
@AudricV AudricV added bug Issue is related to a bug and removed feature request Issue is related to a feature in the app labels Feb 11, 2023
@opusforlife2
Copy link
Collaborator

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.

@Stypox
Copy link
Member

Stypox commented Feb 13, 2023

@SameenAhnaf could you check if this also fixes your "requires two taps" problem?

@SameenAhnaf
Copy link
Collaborator

@Stypox Nope, this PR doesn't fix that bug

@Stypox
Copy link
Member

Stypox commented Feb 14, 2023

@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 WindowCompat if it just causes bugs. I opened #9812.

@Stypox Stypox closed this Feb 14, 2023
@Stypox Stypox reopened this Feb 14, 2023
@Stypox
Copy link
Member

Stypox commented Feb 14, 2023

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 BEHAVIOR_SHOW_BARS_BY_TOUCH which is deprecated on Android 12+.

@Stypox Stypox changed the title Avoid jumping of the player on devices with a cutout. Avoid player jumping on devices with a cutout and fix two taps required for interacting Feb 14, 2023
@Isira-Seneviratne
Copy link
Member Author

@Stypox I simplified one of the methods you added.

@Stypox Stypox mentioned this pull request Feb 14, 2023
6 tasks
@sonarcloud
Copy link

sonarcloud bot commented Mar 3, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@Stypox
Copy link
Member

Stypox commented Apr 3, 2023

Since this still does not solve #9773, I will merge #9812 instead, sorry. Let's not forget about the code in this PR as it might come in handy

@Stypox Stypox closed this Apr 3, 2023
@Isira-Seneviratne Isira-Seneviratne deleted the Fix_fullscreen_jumping branch April 5, 2023 04:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Issue is related to a bug GUI Issue is related to the graphical user interface player Issues related to any player (main, popup and background)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

immersive fullscreen is gone Interaction needs one tap before beeing able to interact
5 participants