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

bug: endscreen cards/ fullscreen panels #248

Closed
3 tasks done
MondayNitro opened this issue Feb 11, 2023 · 3 comments
Closed
3 tasks done

bug: endscreen cards/ fullscreen panels #248

MondayNitro opened this issue Feb 11, 2023 · 3 comments
Labels
Bug report Something isn't working

Comments

@MondayNitro
Copy link

Type

Other

Application

Youtube 18.05.35

Bug description

The swipe up to show endscreen cards gesture does not work if you enable 'hide-fullscreen-panels' and enter fullscreen mode while leaving comments/ live chat open

Steps to reproduce

  1. include hide-fullscreen-panels patch but dont include hide-endscreen-cards patch
  2. play a video, goto fullscreen mode and endscreen cards will show up if you swipe up
  3. now exit fullscreen mode, open comments box or live chat, now go to fullscreen mode again
  4. swipe up as much as you want, endscreen cards wont show up

Relevant log output

none

Screenshots or videos

output.mp4

Solution

bring back hide-fullscreen-buttoncontainer patch. it does the work for most users as the only irritating thing on fullscreen is the button container. as for the fullscreen panels patch, you can remove it or keep it as well. users can choose buttoncontainer patch or panels patch, whichever they like

Additional context

.

Device Environment

LOS 18.1 A11

Acknowledgements

  • I have searched the existing issues and this is a new and no duplicate or related to another open issue.
  • I have written a short but informative title.
  • I filled out all of the requested information in this issue properly.
@MondayNitro MondayNitro added the Bug report Something isn't working label Feb 11, 2023
@inotia00
Copy link
Owner

it's not a bug
hide-fullscreen-panels patch was added by the official ReVanced Team, and if you enable the patch, the tablet layout will only be applied in fullscreen

In the tablet layout, the related video (end screen overlay) is not shown when swiping up.
For the same reason, the tablet layout does not support comment panels in fullscreen
it is Google's intent so it's normal

@MondayNitro
Copy link
Author

even so, can you please bring back the hide-fullscreen-buttoncontainer patch? a lot of people are used to using comments and endscreen cards on fullscreen. i will be grateful if you bring this patch back. i also requested it here #227

@MondayNitro
Copy link
Author

on a different note, patching 18.04.43 gives error

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug report Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants