-
Notifications
You must be signed in to change notification settings - Fork 100
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
Description link colors don't respect current theme #181
Comments
Alright looks like I might have found where the problem is. YouTube has something called 'attributed description' which includes text colors based on the current theme, but the proxy always assumes we are in dark theme. Hence why the description may look weird with the light theme on. @zerodytrash We probably need to get this from the client, but there is no way for me to test if this actually fixes it.
|
`attributedDescription` contains styles (font colors) which depend on the current theme (light and dark). Our payload did not contain the `userInterfaceTheme` property and our proxy server defaulted it to being set to `USER_INTERFACE_THEME_DARK`. Leading to issues where users who have the light theme enabled, were shown text colors which were not readable. Related issues: - #181
Ok, I just figured that I could test this with non age restricted videos and it did work. The fix will be included in the next release. Thanks for reporting. |
This description thing has happened before (I think once when i got this) and it was fixed, either it was automatically fixed or another update. (didn't find out) |
Fixed in 2.5.5 |
"Fixed in 2.5.5" |
The proxy server probably still runs on old code and needs to be updated. @zerodytrash Could we improve the process by auto deploying somehow? |
I thought this was only going to be a server side fix, but we found a bug in the client... @Jamesgsburns Could you try out this build? |
That didn't work for me... Also last night, the sidebar and description were missing again and they just came back but not only did it NOT fix this issue, but that did NOT fix issue #169 either. |
How about this one:
Yes, when updating the server a regression was found, which then resulted in bad responses. This is now resolved.
This was not mean to fix issue #169, that's unrelated. |
Unfortunately that did the exact same thing. It could be because it says it's an unknown publisher, I'm not the best with files. |
If you have Tampermonkey:
|
OMG that worked? Is there any way to update the userscript with that? Keep this issue open for the other people. |
Alright I'll close this issue when the next version is released. |
Fixed in 2.5.6. |
What happened?
I was expecting one of you to notice, it must have been the update.
Platform
Userscript
Version
2.5.4
What browser are you on?
Chrome
Browser version
Present version
Link to the video
https://www.youtube.com/watch?v=ykC7HZ1-FrQ
Screenshots or Videos
About you
The text was updated successfully, but these errors were encountered: