-
-
Notifications
You must be signed in to change notification settings - Fork 227
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: remember auto repeat setting in autorepeat-by-default
patch instead of never turning it off
#2350
Comments
Exclude the auto-repeat patch. |
IMO this is anti-user. What the patch should do is remember the state. So if you toggle it on, it will stay on, if you toggle it off, it will stay off. Simply adding an integrations method to remember the setting would be enough. |
That's not what that patch is for. It clearly says "autorepeat-by-default" and that's exactly what it does. It's toggleable over the setting in the future, also that's why it's excluded by default |
The patch itself is anti-user then. A necessary QoL feature for this patch is required which is that if you toggle it on, it stays on, if you toggle it off, it stays off. (See how you skip the part where you have to toggle it on or off in the settings, which would be the same step?) |
Also this is how it works on YT Desktop. |
I would propose to rename it then |
We can implement that yes. I just did what vanced did without changing anything as for the usage. But we could for sure change it. |
autorepeat-by-default
patch instead of never turning it off
not understand from docs how to disable it |
Then wait until the manager is done. ReVanced is currently not targeted to you then |
This issue is fixed. Closing it now. |
# [2.176.0-dev.6](ReVanced/revanced-patches@v2.176.0-dev.5...v2.176.0-dev.6) (2023-06-12) ### Features * **reddit:** add `hide-promoted` patch ([ReVanced#2350](ReVanced/revanced-patches#2350)) ([c3fd36c](ReVanced/revanced-patches@6a47dc8))
# [2.176.0](ReVanced/revanced-patches@v2.175.0...v2.176.0) (2023-06-12) ### Bug Fixes * **instagram/hide-timeline-ads:** constrain to version `275.0.0.27.98` ([ReVanced#2377](ReVanced/revanced-patches#2377)) ([6ee836c](ReVanced/revanced-patches@7be3f7e)) * **songpal:** make patches more resilient ([ReVanced#2386](ReVanced/revanced-patches#2386)) ([947bf42](ReVanced/revanced-patches@3eaaf4b)) * use correct import ([9b6e730](ReVanced/revanced-patches@92dbd6f)) * **youtube/hide-get-premium:** shorten descriptions ([f5224c4](ReVanced/revanced-patches@e70b2ff)) * **youtube/theme:** use custom light/dark colors for launch splash screen ([ReVanced#2337](ReVanced/revanced-patches#2337)) ([8adc05a](ReVanced/revanced-patches@36a82fc)) ### Features * **citra:** remove patches ([ReVanced#2384](ReVanced/revanced-patches#2384)) ([d182329](ReVanced/revanced-patches@57a98c5)) * **finanzonline:** support any version ([ReVanced#2347](ReVanced/revanced-patches#2347)) ([cc66b1f](ReVanced/revanced-patches@5e9c72a)) * **reddit:** add `hide-promoted` patch ([ReVanced#2350](ReVanced/revanced-patches#2350)) ([c3fd36c](ReVanced/revanced-patches@6a47dc8)) * **songpal:** add `remove-badge-tab` patch ([bde9053](ReVanced/revanced-patches@a300e97)) * **songpal:** add `remove-notification-badge` patch ([1b77f96](ReVanced/revanced-patches@5d7ac6e)) * **syncforreddit/change-oauth-client-id:** improve error message ([a39b2fd](ReVanced/revanced-patches@89d1541)) * **syncforreddit:** add `change-oauth-client-id` patch ([ReVanced#2393](ReVanced/revanced-patches#2393)) ([c93f0c6](ReVanced/revanced-patches@6908373)) * **youtube/hide-ads:** hide mix playlists ([dbb38ed](ReVanced/revanced-patches@0edee9c)) * **youtube/swipe-controls:** add option to save and restore brightness ([3d03471](ReVanced/revanced-patches@8315c5e))
# [2.176.0-dev.6](ReVanced/revanced-patches@v2.176.0-dev.5...v2.176.0-dev.6) (2023-06-12) ### Features * **reddit:** add `hide-promoted` patch ([ReVanced#2350](ReVanced/revanced-patches#2350)) ([c3fd36c](ReVanced/revanced-patches@c3fd36c))
# [2.176.0](ReVanced/revanced-patches@v2.175.0...v2.176.0) (2023-06-12) ### Bug Fixes * **instagram/hide-timeline-ads:** constrain to version `275.0.0.27.98` ([ReVanced#2377](ReVanced/revanced-patches#2377)) ([6ee836c](ReVanced/revanced-patches@6ee836c)) * **songpal:** make patches more resilient ([ReVanced#2386](ReVanced/revanced-patches#2386)) ([947bf42](ReVanced/revanced-patches@947bf42)) * use correct import ([9b6e730](ReVanced/revanced-patches@9b6e730)) * **youtube/hide-get-premium:** shorten descriptions ([f5224c4](ReVanced/revanced-patches@f5224c4)) * **youtube/theme:** use custom light/dark colors for launch splash screen ([ReVanced#2337](ReVanced/revanced-patches#2337)) ([8adc05a](ReVanced/revanced-patches@8adc05a)) ### Features * **citra:** remove patches ([ReVanced#2384](ReVanced/revanced-patches#2384)) ([d182329](ReVanced/revanced-patches@d182329)) * **finanzonline:** support any version ([ReVanced#2347](ReVanced/revanced-patches#2347)) ([cc66b1f](ReVanced/revanced-patches@cc66b1f)) * **reddit:** add `hide-promoted` patch ([ReVanced#2350](ReVanced/revanced-patches#2350)) ([c3fd36c](ReVanced/revanced-patches@c3fd36c)) * **songpal:** add `remove-badge-tab` patch ([bde9053](ReVanced/revanced-patches@bde9053)) * **songpal:** add `remove-notification-badge` patch ([1b77f96](ReVanced/revanced-patches@1b77f96)) * **syncforreddit/change-oauth-client-id:** improve error message ([a39b2fd](ReVanced/revanced-patches@a39b2fd)) * **syncforreddit:** add `change-oauth-client-id` patch ([ReVanced#2393](ReVanced/revanced-patches#2393)) ([c93f0c6](ReVanced/revanced-patches@c93f0c6)) * **youtube/hide-ads:** hide mix playlists ([dbb38ed](ReVanced/revanced-patches@dbb38ed)) * **youtube/swipe-controls:** add option to save and restore brightness ([3d03471](ReVanced/revanced-patches@3d03471))
π Issue
Whenever you play any video, the video loops even when the setting is turned off.
β Reproduce
Built using
Revanced Cli 2.5.3
Revacnced pached 2.13.4
Revanced Integrations 0.24
π Solution
β Additional context
The text was updated successfully, but these errors were encountered: