Skip to content
This repository has been archived by the owner on Oct 26, 2024. It is now read-only.

fix(YouTube - Restore old video quality menu): Do not show error toast if using an old version of YT #640

Merged
merged 2 commits into from
May 27, 2024

Conversation

LisoUseInAIKyrios
Copy link
Contributor

@LisoUseInAIKyrios LisoUseInAIKyrios commented May 24, 2024

On first launch of Android 8.0 (and maybe some slightly newer versions as well), the playback speed and quality menus use whatever is pre-baked into the apk. It's only after a few app restarts does it start using the newer code paths the old quality and custom speed patches expect.

If the speed or quality menu is opened during these initial launches, the menus show the unpatched behavior but also throw exception toasts.

Rather than fix the patch to work for this temporary situation, instead this PR makes the patches more cautious and fallbacks on using the old menus (as if the patches were disabled). After 1 or more app launches the new flyouts are used and the patches then work as expected.

@LisoUseInAIKyrios LisoUseInAIKyrios merged commit 2227b45 into ReVanced:dev May 27, 2024
2 checks passed
@LisoUseInAIKyrios LisoUseInAIKyrios deleted the fix_old_flyout_menu branch May 27, 2024 14:12
revanced-bot pushed a commit that referenced this pull request May 27, 2024
## [1.9.2-dev.2](v1.9.2-dev.1...v1.9.2-dev.2) (2024-05-27)

### Bug Fixes

* **YouTube - Restore old video quality menu:** Do not show error toast if using an old version of YT ([#640](#640)) ([2227b45](2227b45))
revanced-bot pushed a commit that referenced this pull request May 31, 2024
## [1.9.2](v1.9.1...v1.9.2) (2024-05-31)

### Bug Fixes

* **YouTube - Restore old video quality menu:** Do not show error toast if using an old version of YT ([#640](#640)) ([2227b45](2227b45))
* **YouTube - Spoof client:** Disable AV1 if not supported by spoofing an older iOS device ([#644](#644)) ([e28edba](e28edba))
* **YouTube - Spoof client:** Improve Android spoofing ([#641](#641)) ([baf967e](baf967e))
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant