inotia00 / ReVanced_Extended

ReVanced Extended Issues/Request repo
724 stars 29 forks source link

bug: hide-fullscreen-panels #226

Closed MondayNitro closed 1 year ago

MondayNitro commented 1 year ago

Type

Other

Application

YouTube 18.04.41

Bug description

the new hide-fullscreen-panels patch, when used, makes the live chat button in fullscreen player useless. you cant show/hide live chat by tapping the live chat button. The previous hide-fullscreen-buttoncontainer patch was more useful, as it gave you the freedom to keep livechat on/off while this new patch makes the live chat overlay button broken.

Steps to reproduce

step 1 include hide-fullscreen-panels patch step 2 open any livestream in fullscreen step 3 try tapping the live chat button as many times as you want, it wont do anything

Relevant log output

none

Screenshots or videos

photo_6271780342956406160_y

Solution

i suggest to either remove the new patch and bring back the old, or better still, make available both the patches o that users have choice which of the two patches they want to keep. even if the panels patch duplicates some features of buttoncontainer patch, users can simply choose not to include buttoncontainer patch if they include panels patch

Additional context

No response

Device Environment

LOS 18.1 A11

Acknowledgements

inotia00 commented 1 year ago

This is a bug that also exists in the official ReVanced, and there are no plans to fix it.

MondayNitro commented 1 year ago

Don't fix it, just please add back the hide-fullscreen-buttoncontainer patch. Like i mentioned, users can choose which patch they want.

inotia00 commented 1 year ago

then.. write new patch-request I will reflect when I have time later

MondayNitro commented 1 year ago

thanks! suggestion: you can simply include a warning for users in patch description that dont include buttoncontainer patch if youre including hide-panels patch.