style: add dark theme color scheme#7118
Conversation
|
Interesting about Codespaces, they must add something specially to Codespaces to avoid the FOUC, instead of adding it to the OSS repository for some reason. This is really cool! I have not seen |
|
To expand a little on fixing it for VS Code, we serve an HTML file from VS Code so we have no control over it from the outside, so yeah it will have to be fixed in VS Code itself (or patched). Well, I guess we could intercept it and modify it but it seems a bit janky to do it that way. |
https://caniuse.com/mdn-css_types_color_light-dark Yeah, I didn't think much about it since it has become the new baseline and is supported by all browsers.
Well, yes, I agree with you. I'm not familiar with those structures, so perhaps need more time to investigate. So maybe another PR that should contains:
I think having these will greatly improve the first experience for users prefer a dark theme. #3019 would be nice, but that's more of a icing on the cake. |
When the user's browser does not support `light-dark`, fall back to the light theme color scheme.
|
Agreed with your comments! Thanks again, this is great. |
hen the user's browser does not support `light-dark`, fall back to the light theme color scheme.
Partially fixes the issue mentioned in #7077.
Using vscode theme colors is proposed in #3019, that would be nice. But until it is implemented, it might still be helpful as a fallback for browser adaptation in the absence of user preferences.
The problem is that this PR only solves the color scheme of login page, error pages.
I haven't found out how to modify the vscode page, so the FOUC is still there.
It seems that vscode page needs to be modified in
lib/vscode? I.e. is this also an upstream issue?I tested https://vscode.dev and it seems to have the same FOUC issue.
Or should actually compare to
Codespace(https://github.dev) , which actually has no FOUC issues, in my testing.