inotia00 / ReVanced_Extended

ReVanced Extended Issues/Request repo
712 stars 29 forks source link

feat(YouTube Music/Amoled): add patch options #2440

Open innuendofloyd opened 2 hours ago

innuendofloyd commented 2 hours ago

Application

YouTube Music

Feature description

Hello, my eyes really hurt when I read white fonts on a amoled/black background, so I usually theme color.xml. The recent Navbar patch overwrite my (kinda) theme. Even if I disable black Navbar it stays greysh. I kindly request to allow the choice of Navbar color, in options file maybe? NEW OLD

Thank you.

Motivation

I know it could be "ad personam" request, I just like the UI to be neat with the color I chose. My eyes are thankful with #FF5b513f , so I don't see white fonts in the sky for an half hour after I used the app. Thank you anyway

Acknowledgements

inotia00 commented 2 hours ago

please add the color name of the colors.xml you modified

(e.g. "yt_black0", "yt_black1", ...)

innuendofloyd commented 2 hours ago

Since I'm a butcher and not a developer I basically changed every black or dark color in colors.xml colors.zip

inotia00 commented 2 hours ago

In this case, it would be difficult to implement with a standardized patch

innuendofloyd commented 2 hours ago

I don't demand a total theme for the entire app. Maybe could it be possible to separate the black navbar code from the navbar patch? My point is I need it just to hide "sample" and "upgrade to premium" Moreover, could I exclude navbar patch totally, and hide "sample" and "premium" in another way?

inotia00 commented 2 hours ago

It's hard to separate only the Enable black navigation bar patch from the Navigation bar components patch

Instead, I can make it use the color defined in colors.xml (@color/ytm_color_grey_12) instead of the hardcoded color when the Enable black navigation bar setting is off

innuendofloyd commented 2 hours ago

Yes!! Thank you!!