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

feat(YouTube): Support versions 19.25 and 19.34 #3629

Open
wants to merge 101 commits into
base: dev
Choose a base branch
from

Conversation

zainarbani
Copy link
Contributor

@zainarbani zainarbani commented Sep 7, 2024

Adds support for versions 19.25.38 and 19.34.42

Except for the Miniplayer patch, all other targets between 19.16 and 19.34 seem to work without any issues. But for simplicity, there is only 2 new recommended versions.
 

Recommended app version

Because of the possibility of unidentified issues and the lack of Restore old seekbar thumbnails support, the recommended app version is still 19.16.39

Bugs, stability

The recent versions of YouTube have many A/B tests. Both of these new versions may experience more bugs and partially broken patch features. It's also possible you may encounter unidentified ads compared to using 19.16.
 

Seekbar thumbnails

Restore old seekbar thumbnails is not supported on any version after 19.16.39. The code for the old style thumbnails is no longer present and there is no way to restore. If you want the old seekbar thumbnails then continue patching 19.16.39.
 

Miniplayer

The modern miniplayers are still under development by YouTube, and as such there can be bugs and issues while using them.

The later targets both add and remove some miniplayer features.

  • 19.25.37 adds miniplayer drag and drop and double tap and pinch to increase the miniplayer size. It is also the last version that can expand/close the miniplayer by swiping (must have both drag and drop and double tap turned off), and the last version with 5 second skip forward/back buttons on Modern 1.
  • 19.34.42 The initial miniplayer size can be specified in the settings. Pinch to resize is always on (cannot be turned off).

 

TODO

  • Refactor LithoFilter patch (method changed).
  • Finish and merge fix(YouTube): Fix issues related to playback by replace streaming data #3582 (this PR is dependent on those changes).
  • Fix set start page patch
  • Fix why some litho components are inconsistently hiding.
  • Verify 18.48 thru 19.29 work.
  • Fix SponsorBlock vote & create segment buttons functionality (not working and always shown on player).
  • Fix up Miniplayer feature flags, pick a few targets that work well to use for version recommendations
  • Fix up Enable slide to seek
  • Fix scrolling broken after minimizing video when using 'original' miniplayer

Integration changes

@LisoUseInAIKyrios
Copy link
Contributor

All patches are version bumped, except for Restore old seekbar thumbnails?

The YT code for old seekbar thumbnails is removed after 19.16, so there's no way a patch can force it on.

Do any versions between 19.16 and 19.31 also work?

@LisoUseInAIKyrios
Copy link
Contributor

LisoUseInAIKyrios commented Sep 7, 2024

Since old seekbar patch will remain at 19.16, that means the recommended version will still show as 19.16, but I think for now that is good.

There is many changes and so many a/b tests in the recent YT versions that it's almost certain some users will experience half broke patch functionality or run into new annoying content that has no removal patch. So it's better to recommending 19.16 unless a user wants to opt in to brave patching the latest.

@zainarbani

This comment was marked as resolved.

@LisoUseInAIKyrios

This comment was marked as resolved.

@zainarbani

This comment was marked as resolved.

@LisoUseInAIKyrios

This comment was marked as resolved.

@LisoUseInAIKyrios

This comment was marked as outdated.

@LisoUseInAIKyrios

This comment was marked as resolved.

@LisoUseInAIKyrios LisoUseInAIKyrios changed the title feat(YouTube): Support version ~19.31 feat(YouTube): Support version ~19.30 Sep 8, 2024
@zainarbani

This comment was marked as resolved.

@LisoUseInAIKyrios LisoUseInAIKyrios changed the title feat(YouTube): Support versions 19.25 and 19.37 feat(YouTube): Support versions 19.25 and 19.34 Sep 25, 2024
…-bump

# Conflicts:
#	src/main/kotlin/app/revanced/patches/youtube/layout/hide/shorts/HideShortsComponentsResourcePatch.kt
@zainarbani
Copy link
Contributor Author

@LisoUseInAIKyrios
Copy link
Contributor

@zainarbani I have Premium and I've never seen that. But I don't ever remember seeing any new features offered to my account.

@zainarbani
Copy link
Contributor Author

@zainarbani I have Premium and I've never seen that. But I don't ever remember seeing any new features offered to my account.

Same, Just looking for "Ask about this video" on decompiled 19.38 and nothing matches 😅

@cyberboh
Copy link

Same, Just looking for "Ask about this video" on decompiled 19.38 and nothing matches 😅

It could be server-sided, because A.I is generated via server.

@LisoUseInAIKyrios
Copy link
Contributor

I think this is ready to merge.

But at the same time, there is no hurry to merge since there isn't anything different with 19.34 except the seekbar thumbnails is worse.

I've gone back to using 19.16 just to have old seekbar and swipe to expand/close Modern 1 miniplayer.

@zainarbani
Copy link
Contributor Author

I don't even understand why youtube remove the old seekbar and introduces unused features like smart download

@zainarbani
Copy link
Contributor Author

@LisoUseInAIKyrios @0xrxL Any idea why cairo settings behave this way?

ss

Screenshot_2024-09-27-20-14-58-119_app revanced android youtube-edit

@0xrxL
Copy link

0xrxL commented Sep 27, 2024

@LisoUseInAIKyrios @0xrxL Any idea why cairo settings behave this way?

ss

If you referring to missing icons, I think because it's still under development.

But seems you found a way to add revanced settings button to it. Great...

@zainarbani
Copy link
Contributor Author

But seems you found a way to add revanced settings button to it. Great...

Yeah the missing icons, but i think i have completed icons on my previous premium account.

@0xrxL
Copy link

0xrxL commented Sep 27, 2024

But seems you found a way to add revanced settings button to it. Great...

Yeah the missing icons, but i think i have completed icons on my previous premium account.

Good...post a screen here when you can. 👍

@LisoUseInAIKyrios
Copy link
Contributor

Allowing (or forcing) cairo settings breaks some other patches such as custom seekbar color and I think background playback.

@LisoUseInAIKyrios LisoUseInAIKyrios linked an issue Sep 27, 2024 that may be closed by this pull request
3 tasks
…StandalonePlayerActivityFingerprint` in IntegrationsPatch). Verified relative seek works correctly with tv casting.
@0xrxL
Copy link

0xrxL commented Sep 28, 2024

@zainarbani Google bugged this on manifest, which is the cause of light splashscreen that hurt eyes 🩻

Immagine 2024-09-28 094536

Just use a different style to fix it 👍

@zainarbani
Copy link
Contributor Author

@0xrxL I have no idea about UI stuff tbh 😅

@0xrxL
Copy link

0xrxL commented Sep 28, 2024

@0xrxL I have no idea about UI stuff tbh 😅

Just in case someone will open an issue related to the "flashing white splashscreen".

@inotia00
Copy link

@zainarbani Google bugged this on manifest, which is the cause of light splashscreen that hurt eyes 🩻

Immagine 2024-09-28 094536 Just use a different style to fix it 👍

As I mentioned in ReVanced/revanced-patches-template#2178 (comment), modifying AndroidManifest does not apply to root users

@0xrxL
Copy link

0xrxL commented Sep 28, 2024

@zainarbani Google bugged this on manifest, which is the cause of light splashscreen that hurt eyes 🩻
Immagine 2024-09-28 094536
Just use a different style to fix it 👍

As I mentioned in ReVanced/revanced-patches-template#2178 (comment), modifying AndroidManifest does not apply to root users

Neither style.xml? Because this fix can be applied to the bugged declared (and used by Manifest) style.

@inotia00
Copy link

only manifest modification is impossible

changes via style.xml are possible

…-bump

# Conflicts:
#	src/main/kotlin/app/revanced/patches/youtube/interaction/seekbar/DisablePreciseSeekingGesturePatch.kt
#	src/main/kotlin/app/revanced/patches/youtube/interaction/seekbar/fingerprints/IsSwipingUpFingerprint.kt
#	src/main/kotlin/app/revanced/patches/youtube/layout/hide/shorts/HideShortsComponentsResourcePatch.kt
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment