-
-
Notifications
You must be signed in to change notification settings - Fork 310
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): Improve patch descriptions #1054
Comments
The descriptions should be short, can you please suggest short descriptions for every of the toggles you mentioned? |
The optional YouTube patches that do not have toggles are:
That's 5 patches. Not counting those 5, there are about 50 other patches that have in app setting switches. That means nearly every patch would say "adds a toggle for". |
Looking at the descriptions one could either say "Toggle that ----" or it could be written like the Hide player buttons "Adds the option to hide video player previous and next buttons." Example with disable-auto-captions "Disable forced captions from being automatically enabled."
|
If anything, just those 5 patches that are permanent (no way to turn on/off inside the app) should explain they are on all the time. |
@LisoUseInAIKyrios I believe the issue is in the descriptions of the toggle preferences. They just say "enabled" or "disabled" expressing the state of the toggle, whereas the description is meant to describe the toggle, not just express its state. |
It would be great if the patches could declare themselves as configurable. And the manager then automatically included and hides the selection of these patches from the user during patching (in other words, they get the patch no matter what) So the user only needs to decide which of the the permanent patches to include. Which right now is just 5 patches. Of course, let the user manually include/exclude patches if they wish, but make that an 'expert' mode or something to that effect. It would dramatically simplify patching, which is intimidating to users when they're asked to pick from 50 patches they don't easily understand. It would dramatically reduce the number of users who don't follow guides and the users don't use the recommended patches, and then they post the usual "my videos won't play after 30 seconds omg what is this broke app I picked spoof client and it's still broken, what is spoof verification help me omg" |
Yes, this is in ongoing discussion in ReVanced Manager repository. The idea is to promote the default selection of patches but allow the user to alter it if they really want to. |
In that case, maybe just add "allow" Hide shorts button -> Allow hiding the shorts button |
The problem is something like "Hide product banner" does not explain what the product banner is. People don't know what they are hiding. |
Maybe do a screenshot example on what that actually hides |
There is no room for screenshots in the components and design language that YouTube uses. |
I mean in some documentation repo |
All documentation is agnostic to any patch. At max patches can document itself |
I'd like to PR for this (and #2515) I think a good wording would be I'm not sure if this should be added to every patch that is toggleable . Some patches are essentially necessary and shouldn't be disabled, such as the |
As long as you are consistent in wording and terminology, repetition shouldn't be a problem |
Application
Youtube
Issue
Some of the patches add a toggle but in the description they don't mention it, they just say it removes stuff
Patch
Description should say "Adds a toggle that ____" (or any variation depending on the context)
This is a list of some of the patches that i use that have a toggle but don't mention them, there's probably more but i don't want to repatch my current apk just to test
Motivation
It will make it easy to know what adds a toggle and what simply just works.
Acknowledgements
The text was updated successfully, but these errors were encountered: