feat(YouTube Music): Add Settings patch
#2708
Conversation
| <string name="revanced_debug_toast_on_error_summary_on">Toast shown if error occurs</string> | ||
| <string name="revanced_debug_toast_on_error_summary_off">Toast not shown if error occurs</string> | ||
| <string name="revanced_debug_toast_on_error_user_dialog_message">Turning off error toasts hides all ReVanced | ||
| error notifications.\n\nYou will not be notified of any unexpected events. |
There was a problem hiding this comment.
When the strings were extracted a month ago, I noticed that if the strings have line wrap spacing and new line characters (actual new line, and not \n), then the strings incorrectly show up in the settings UI with some of that formatting.
I did not take a screen shot of it, but every summary string with \n would incorrectly show a space or two between the \n and the text after it.
Removing the extra line wrapping and indentation fixed the summary text layout.
It's not so pretty, but using no line wrapping may be needed.
There was a problem hiding this comment.
Maybe it's just an issue with using \n character with indentation immediately before or after it. Maybe it'll work if there's only 1 indentation after the tag such as:
<string name="revanced_debug_toast_on_error_user_dialog_message">
Turning off error toasts hides all ReVanced error notifications.\n\nYou will not be notified of any unexpected events.</string>
But I haven't looked into this.
oSumAtrIX
force-pushed
the
dev
branch
3 times, most recently
from
d964b8f
to
fd27725
Compare
# Conflicts: # src/main/kotlin/app/revanced/patches/youtube/misc/debugging/DebuggingPatch.kt # src/main/kotlin/app/revanced/patches/youtube/misc/settings/SettingsResourcePatch.kt # src/main/resources/addresources/values/strings.xml
|
In YTM v6.43+, Google removed |
About
WsaClient_ukEwU1uHCr.mp4
This PR adds a patch that adds ReVanced settings to YouTube Music.
This is achieved through recent abstractions regarding settings and resource patches.
The PR works like this:
Add a new settings patch, similar to YouTube, to YouTube Music. It extends existing abstract classes.
Because
BaseSettingsin integrations has to debug strings by default, the Debugging patch was abstracted in a similar fashion to aBaseDebuggingPatch. YouTube adds an additional debugging preference (Litho buffer logging), which does not exist in YouTube Music yet. Refer to the current implementation.RE: ReVanced/revanced-integrations#568
Todo
SettingsPatchtoBaseSettingsPatchsimilar toBaseSettingsResourcePatchThemeHelper" which is currently responsible in YouTube for selecting the correct theme for the settings screensThemeHelperin YouTube MusicThemeHelpermay be merged intoSettingsPatchin some way or another; it is exclusively used for that purpose, IIRCstrings.xml, maybe fix them, but IDEs have the option to hide such changes in the diff viewer: