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 (YouTube): Hide info panels hides panel that should not be hidden #3245

Open
3 tasks done
KobeW50 opened this issue May 29, 2024 · 9 comments
Open
3 tasks done

bug (YouTube): Hide info panels hides panel that should not be hidden #3245

KobeW50 opened this issue May 29, 2024 · 9 comments
Labels
Bug report Something isn't working

Comments

@KobeW50
Copy link
Contributor

KobeW50 commented May 29, 2024

Bug description

App/Patches info

App: YouTube v19.11.43
Patches: v4.8.2

Bug details

The Hide info panels option (found in Settings > ReVanced > Player) also hides the following section on the page that appears when you search about "suicide".

Screenshot_20240529_150840

Why this is a bug

This panel should not be hidden for 2 reasons:

  1. The Hide info panels option is found in the Player settings, so it should only hide panels in the player.

  2. When the panel (shown above) is hidden, it is impossible to view the search results. So by hiding it, it is only an annoyance rather than a useful feature. At the very least, there should be a warning about this downside in the toggle description.

Error logs

No response

Solution

No response

Additional context

No response

Acknowledgements

  • This issue is not a duplicate of an existing bug report.
  • I have chosen an appropriate title.
  • All requested information has been provided properly.
@KobeW50 KobeW50 added the Bug report Something isn't working label May 29, 2024
@LisoUseInAIKyrios
Copy link
Contributor

Does this issue still show up? I cannot reproduce it (it never shows the "you are not alone" warning).

@KobeW50
Copy link
Contributor Author

KobeW50 commented Aug 25, 2024

The issue still occurs on YT 19.16.39 with patches v4.13.3

I'm guessing the warning only appears in countries where there that suicide hotline functions

@LisoUseInAIKyrios
Copy link
Contributor

LisoUseInAIKyrios commented Aug 25, 2024

I was able to reproduce it on a different account.

It appears filtering just |information_panel.eml hides 2019 disease warnings, and the 'show anyways' is not hidden as desired.

The | is needed for the filter to not hide single_item_information_panel.eml (which incorrectly hides the 'show anyways' button).

Not sure if this change works for other usage of info panel, because these two use cases are all I can find where it pops up.

This setting should also be moved to General since it shows up in the feed and in the player.

Edit: some additional filters are needed to hide some components that are no longer filtered, such as publisher_transparency_panel.eml. there is likely additional unidentified components as well.

@0xrxL
Copy link

0xrxL commented Oct 6, 2024

@KobeW50 It's just a test code but...try it anyway:

Define a variable outside method:

private static int frictionPanelState = 0;

Add the following in the method instead:

//---------------------------------Friction Panel------------------------------------//
        if (combinedTemplates.contains("cell_divider")) {
            frictionPanelState = 1;
        } else if (combinedTemplates.contains("single_item_information_panel")) {
            if (frictionPanelState == 1) {
                frictionPanelState = 2;
            }

            return frictionPanelState == 2;
        } else if (!combinedTemplates.contains("null") &&
                    !combinedTemplates.contains("search_friction")) {
                        frictionPanelState = 0;
        }
        //-----------------------------------------------------------------------------------//

@KobeW50
Copy link
Contributor Author

KobeW50 commented Oct 6, 2024

It's just a test code but...try it anyway:

Due to time contraints and lack of knowledge I'm unable to make my own patches to test with. Thank you for your efforts. Sorry

@0xrxL
Copy link

0xrxL commented Oct 6, 2024

It's just a test code but...try it anyway:

Due to time contraints and lack of knowledge I'm unable to make my own patches to test with. Thank you for your efforts. Sorry

It works on my side anyway...

This is how it appears in the search view

Image

inotia00 added a commit to inotia00/revanced-patches that referenced this issue Feb 13, 2025
@LisoUseInAIKyrios
Copy link
Contributor

I cannot get this warning to show anymore.

@KobeW50
Copy link
Contributor Author

KobeW50 commented Feb 17, 2025

It still shows for me on 19.47.53

@LisoUseInAIKyrios
Copy link
Contributor

I was able to reproduce only once then not again. It should be fixed with 4ac8854 but I cannot completely confirm.

@LisoUseInAIKyrios LisoUseInAIKyrios linked a pull request Feb 18, 2025 that will close this issue
1 task
@LisoUseInAIKyrios LisoUseInAIKyrios removed a link to a pull request Feb 19, 2025
1 task
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

3 participants