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 Shorts components): Subscribe button is not hidden #3030

Closed
3 tasks done
ILoveOpenSourceApplications opened this issue Apr 17, 2024 · 22 comments · Fixed by ReVanced/revanced-integrations#603
Labels
Bug report Something isn't working

Comments

@ILoveOpenSourceApplications
Copy link
Contributor

ILoveOpenSourceApplications commented Apr 17, 2024

Bug description

Hide subscribe button no longer hides it in shorts. Meanwhile, turning on Hide channel bar is hiding it in shorts. I have my doubts that this could be an A/B testing as when I went into incognito mode and played the same shorts, there wasn't any subscribe button.

Settings Outcome
Screenshot_2024-04-17-13-04-23-989_app revanced android youtube Screenshot_2024-04-17-13-05-52-701_app revanced android youtube

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.
@ILoveOpenSourceApplications ILoveOpenSourceApplications added the Bug report Something isn't working label Apr 17, 2024
@LisoUseInAIKyrios
Copy link
Contributor

It looks like it's part of the channel bar and an a/b test.

Turn on debug and protobuffer logging and post the log entries for the channel bar and/or subscribe button.

@LisoUseInAIKyrios LisoUseInAIKyrios changed the title bug (YouTube/Hide subscribe button): Subscribe button isn't hidden for Shorts bug(YouTube - Hide Shorts components): Subscribe button isn't hidden Apr 17, 2024
@LisoUseInAIKyrios LisoUseInAIKyrios changed the title bug(YouTube - Hide Shorts components): Subscribe button isn't hidden bug(YouTube - Hide Shorts components): Subscribe button is not hidden Apr 17, 2024
@ILoveOpenSourceApplications
Copy link
Contributor Author

It looks like it's part of the channel bar and an a/b test.

Yes it's indeed a part of channel bar as toggling on hide channel bar hides the subscribe button as well.

@ILoveOpenSourceApplications
Copy link
Contributor Author

04-17 14:45:16.370 19992 19992 D revanced: LithoFilterPatch: Searching ID: null Path: reel_metapanel.eml|510ea3c8ab26c965|ContainerType|reel_channel_bar.eml|282c120d9965d55e|reel_channel_bar_inner.eml|cb7965c5020198c7|ContainerType| BufferStrings: Key_ReelChannelBarInnerWithControllerΓ¥Öreel_subscribe_buttonΓ¥ÖKey_SubscribeButtonWithControllerΓ¥Ösubscribe_button.eml|5479dbde336bb700Γ¥Ö@mkbhdΓ¥Ösans-serif-mediumΓ¥ÖUCBJycsmduvYEL83R_U4JriQΓ¥Ö8gYKCgiaAQCiAQIIAQ%3D%3D"Γ¥Ö/@mkbhdΓ¥Ödecorated_avatar.eml|5c6ca1f502890e70Γ¥ÖUCBJycsmduvYEL83R_U4JriQΓ¥Ö8gYKCgiaAQCiAQIIAQ%3D%3D"Γ¥Ö/@mkbhdΓ¥Ö

@LisoUseInAIKyrios
Copy link
Contributor

Does subscribe_button.eml appear in any log statements after that?

@ILoveOpenSourceApplications
Copy link
Contributor Author

04-17 14:45:16.379 19992 19992 D revanced: LithoFilterPatch: Searching ID: null Path: reel_metapanel.eml|510ea3c8ab26c965|ContainerType|reel_channel_bar.eml|282c120d9965d55e|reel_channel_bar_inner.eml|cb7965c5020198c7|ContainerType|ContainerType|ContainerType|subscribe_button.eml|5479dbde336bb700| BufferStrings: Key_SubscribeButtonWithControllerΓ¥Öcompact_subscribe_button.eml|74c4ba50b9baf501Γ¥Ö

@LisoUseInAIKyrios
Copy link
Contributor

For now a custom filter of subscribe_button.eml might hide it.

The patch can do additional checks like only hiding if the path also starts with reel_metapanel.eml

@ILoveOpenSourceApplications
Copy link
Contributor Author

ILoveOpenSourceApplications commented Apr 17, 2024

For now a custom filter of subscribe_button.eml might hide it.

This hides the subscribe button for the entire YouTube layout though, right?

@LisoUseInAIKyrios
Copy link
Contributor

It'll hide it everywhere else that uses that same path component name, yeah. Not sure where else that is. A custom filter would only be temporarily.

@LisoUseInAIKyrios
Copy link
Contributor

Try and see if that hides it in Shorts.

@ILoveOpenSourceApplications
Copy link
Contributor Author

Try and see if that hides it in Shorts.

It does, but it also hides the subscribe button in the channel page and under a normal video as well, Pretty much everywhere, where there is a subscribe button.

@ILoveOpenSourceApplications
Copy link
Contributor Author

Can't we try a litho$buffer string combo here?

@LisoUseInAIKyrios
Copy link
Contributor

If filter only the path works, it's more efficient to use that.

Does filtering subscribe_button.eml hide the button?

@ILoveOpenSourceApplications
Copy link
Contributor Author

Yes.

@ILoveOpenSourceApplications
Copy link
Contributor Author

It does, but it also hides the subscribe button in the channel page and under a normal video as well, Pretty much everywhere, where there is a subscribe button.

.

@ILoveOpenSourceApplications
Copy link
Contributor Author

Found it. compact_subscribe_button.eml only hides it for shorts.

@LisoUseInAIKyrios
Copy link
Contributor

I just made a change on dev using the reel_metapanel.eml and subscribe_button.eml check.

Try patching with the dev release when it finishes building, and see if that fixes it.

@ILoveOpenSourceApplications
Copy link
Contributor Author

Did you test and see if it doesn't hide it for the general layout?

@LisoUseInAIKyrios
Copy link
Contributor

It won't hide for general layout because it only filters if the path also starts with reel_metapanel.eml (which is a Shorts specific component)

@ILoveOpenSourceApplications
Copy link
Contributor Author

How long does it take for patching with CLI?
image

@LisoUseInAIKyrios
Copy link
Contributor

About 1 minute.

Be sure you have Zulu or OpenJDK described here:
https://github.com/ReVanced/revanced-cli/blob/main/docs/0_prerequisites.md

Use java -version to verify you are running the correct java (and not some other in the command path)

@ILoveOpenSourceApplications
Copy link
Contributor Author

Use java -version to verify you are running the correct java (and not some other in the command path)

D:\Files\platform-tools\CLI>java -version
openjdk version "11.0.23" 2024-04-16 LTS
OpenJDK Runtime Environment Zulu11.72+19-CA (build 11.0.23+9-LTS)
OpenJDK 64-Bit Server VM Zulu11.72+19-CA (build 11.0.23+9-LTS, mixed mode)

@ILoveOpenSourceApplications
Copy link
Contributor Author

Yes, it works @LisoUseInAIKyrios.

@LisoUseInAIKyrios LisoUseInAIKyrios linked a pull request Apr 17, 2024 that will close this issue
1 task
validcube pushed a commit to validcube/revanced-patches that referenced this issue Aug 6, 2024
validcube pushed a commit to validcube/revanced-patches that referenced this issue Aug 6, 2024
# [2.191.0](ReVanced/revanced-patches@v2.190.0...v2.191.0) (2023-10-04)

### Bug Fixes

* Add hooks to existing hook set ([5655067](ReVanced/revanced-patches@5655067))
* **Duolingo - Unlock Super:** Get correct instruction offset ([ReVanced#3023](ReVanced/revanced-patches#3023)) ([5146de8](ReVanced/revanced-patches@5146de8))
* **Google Recorder - Remove device restrictions:** Clarify limitation ([094f57b](ReVanced/revanced-patches@094f57b))
* **Infinity for Reddit - Spoof client:** Constrain patch to last working versions ([ReVanced#2944](ReVanced/revanced-patches#2944)) ([ee975de](ReVanced/revanced-patches@ee975de))
* **Relay - Spoof client:** Restore OAuth login ([96e01f7](ReVanced/revanced-patches@96e01f7))
* **Slide - Spoof client:** Use correct patch name ([f355dbf](ReVanced/revanced-patches@f355dbf))
* **Sync for Reddit:** Do not throw an error when not necessary ([ef644e4](ReVanced/revanced-patches@ef644e4))
* **TikTok - Hide ads:** Constrain to last working version ([56e45a6](ReVanced/revanced-patches@56e45a6))
* **Twitch - Audio ads:** Support missing version `16.1.0` ([ReVanced#2928](ReVanced/revanced-patches#2928)) ([688d8fa](ReVanced/revanced-patches@688d8fa))
* Use consistent toggle description ([ReVanced#2983](ReVanced/revanced-patches#2983)) ([ceaa512](ReVanced/revanced-patches@ceaa512))
* Use correct instruction ([246cf2c](ReVanced/revanced-patches@246cf2c))
* **YouTube - Client spoof:** adjust settings text ([ReVanced#3035](ReVanced/revanced-patches#3035)) ([59a2e96](ReVanced/revanced-patches@59a2e96))
* **YouTube - Client spoof:** Display seekbar thumbnails in high quality ([5e8a2d3](ReVanced/revanced-patches@5e8a2d3))
* **YouTube - Client spoof:** Do not record feed videos to history by default ([ReVanced#3017](ReVanced/revanced-patches#3017)) ([5ccbf1b](ReVanced/revanced-patches@5ccbf1b))
* **YouTube - Client spoof:** Removed unused code ([ReVanced#3030](ReVanced/revanced-patches#3030)) ([15e27bf](ReVanced/revanced-patches@15e27bf))
* **YouTube - Client spoof:** Restore seekbar thumbnails ([bf4a115](ReVanced/revanced-patches@bf4a115))
* **YouTube - Client spoof:** Show seekbar thumbnail for age restricted and paid videos ([1a79300](ReVanced/revanced-patches@1a79300))
* **YouTube - Custom filter:** Use new lines between components instead of commas ([ReVanced#2952](ReVanced/revanced-patches#2952)) ([ecb2e32](ReVanced/revanced-patches@ecb2e32))
* **YouTube - Hide "Load more" button:** Use correct names ([569c3cd](ReVanced/revanced-patches@569c3cd))
* **YouTube - Hide info cards:** Fix info cards not hiding for some users ([ReVanced#3039](ReVanced/revanced-patches#3039)) ([cb38637](ReVanced/revanced-patches@cb38637))
* **YouTube - Hide layout components:** Always hide redundant 'player audio track' button ([ReVanced#2951](ReVanced/revanced-patches#2951)) ([ca632bd](ReVanced/revanced-patches@ca632bd))
* **YouTube - Hide shorts components:** Fix hiding navigation bar ([2de51e6](ReVanced/revanced-patches@2de51e6))
* **YouTube - Premium heading:** Correct inverted logic ([ReVanced#3042](ReVanced/revanced-patches#3042)) ([b33ed75](ReVanced/revanced-patches@b33ed75))
* **YouTube - ReturnYouTubeDislike:** Revert support for 18.37.36 ([ReVanced#3041](ReVanced/revanced-patches#3041)) ([3761073](ReVanced/revanced-patches@3761073))
* **YouTube - SponsorBlock:** Adjust import/export UI text ([ReVanced#3063](ReVanced/revanced-patches#3063)) ([4e5513e](ReVanced/revanced-patches@4e5513e))
* **YouTube - Video Id:** Fix video id not showing the currently playing video ([ReVanced#3038](ReVanced/revanced-patches#3038)) ([f6f226b](ReVanced/revanced-patches@f6f226b))
* **YouTube:** Restore functionality of `Old video quality menu` and `Custom speeds` on tablets ([ReVanced#2999](ReVanced/revanced-patches#2999)) ([238bed1](ReVanced/revanced-patches@238bed1))

### Features

* Add notice for thumbnails for age restricted or paid videos ([f7cf70b](ReVanced/revanced-patches@f7cf70b))
* **Duolingo:** Remove `Unlock Duolingo Super` patch ([b4b9746](ReVanced/revanced-patches@b4b9746))
* **SPB Serviceportal Bund:** Add `Remove root detection` patch ([ReVanced#3049](ReVanced/revanced-patches#3049)) ([481bf58](ReVanced/revanced-patches@481bf58))
* **Strava:** Add `Disable subscription suggestions` patch ([ReVanced#2997](ReVanced/revanced-patches#2997)) ([af02175](ReVanced/revanced-patches@af02175))
* **Sync for Reddit - Client spoof:** Restore upload functionality ([9344c8a](ReVanced/revanced-patches@9344c8a))
* **TikTok - Hide ads:** Constrain to last working version ([516e8a1](ReVanced/revanced-patches@516e8a1))
* **TU Dortmund:** Add `Show on lockscreen` patch ([ReVanced#2947](ReVanced/revanced-patches#2947)) ([9a18326](ReVanced/revanced-patches@9a18326))
* **Tumblr:** Add `Disable blog notification reminder` patch ([29f19b9](ReVanced/revanced-patches@29f19b9))
* **Tumblr:** Add `Disable dashboard ads` patch ([ReVanced#2979](ReVanced/revanced-patches#2979)) ([07c267a](ReVanced/revanced-patches@07c267a))
* **Tumblr:** Add `Disable gift message popup` patch ([a4d6b4e](ReVanced/revanced-patches@a4d6b4e))
* **Tumblr:** Add `Disable in-app update` patch ([ReVanced#3058](ReVanced/revanced-patches#3058)) ([5e8076b](ReVanced/revanced-patches@5e8076b))
* **Tumblr:** Add `Disable Tumblr Live` patch ([ReVanced#2987](ReVanced/revanced-patches#2987)) ([bf1f9dc](ReVanced/revanced-patches@bf1f9dc))
* **Twitch - Block embedded ads:** Switch from `ttv.lol` to `luminous.dev` ([0fe115e](ReVanced/revanced-patches@0fe115e))
* Use properties file for `Client spoof` patches ([e5d548c](ReVanced/revanced-patches@e5d548c))
* **YouTube - Hide layout components:** Hide "Join" button ([1b71f89](ReVanced/revanced-patches@1b71f89))
* **YouTube - Hide layout components:** Hide "Notify me" button ([3027c15](ReVanced/revanced-patches@3027c15))
* **YouTube - Hide layout components:** Hide search result shelf header ([f4eda8c](ReVanced/revanced-patches@f4eda8c))
* **YouTube - Hide layout components:** Hide timed reactions ([d0a775d](ReVanced/revanced-patches@d0a775d))
* **YouTube - Premium heading:** Allow using default heading ([ReVanced#3029](ReVanced/revanced-patches#3029)) ([d5ab35a](ReVanced/revanced-patches@d5ab35a))
* **YouTube - Spoof app version:** add version 18.20.39 ([ReVanced#3001](ReVanced/revanced-patches#3001)) ([f14c5e7](ReVanced/revanced-patches@f14c5e7))
* **YouTube:** Add `Bypass URL redirects` patch ([125cac5](ReVanced/revanced-patches@125cac5))
* **YouTube:** Bump compatibility to `18.37.36` ([ReVanced#3028](ReVanced/revanced-patches#3028)) ([eda28e5](ReVanced/revanced-patches@eda28e5))
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

Successfully merging a pull request may close this issue.

2 participants