-
-
Notifications
You must be signed in to change notification settings - Fork 156
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) Enabling ads doesn't enable ads (solution depends on changing another setting) #3103
Comments
It's not a bug |
The hide ads setting can show a dialog message if the user tries to turn it off. |
It may not be a bug, but it makes the enable ads setting useless if you don't turn it off. My issue is that it isn't clear that you must turn it off. I certainly had no idea that this setting was blocking ads before a lot of trial and error. The setting description itself also doesn't mention that it blocks ads, so even if someone finds this setting, which isn't in the ads menu, they would have no idea they have to toggle it or that it has any relation to ads at all. |
@LisoUseInAIKyrios both patches are separate so the ads the patch can't display a dialog about the other patch since it doesn't know anything about it |
Yeah that is true. I suppose the best (simple) solution is to add a text description to the switch notifications setting summary. |
Maybe you could change the text under “switch create with notifications” to say that it also disables video ads? That way at least it’s visible somewhere and makes the description more accurate. |
Bug description
I was having issues enabling video ads in revanced. Note that I get a lot of ads on my phone and my laptop so I don't think its because of where I live or smthg.
What I tried
Error logs
No response
Solution
Eventually I found out that if you turn off
it works again.
I suggest that this setting is automatically turned off when someone disables video ads, or that the person is given a popup telling them to do so, so that they aren't confused when ads don't appear anyways.
Additional context
No response
Acknowledgements
The text was updated successfully, but these errors were encountered: