Open shreyaswikriti opened 3 years ago
Having dark code snippets seems to break a bit with the rest of the site, which is light, doesn't it? Do you have an example of a site where that happens?
it's the website of the iconic framework. They also use dark extension. Actually, I thought this extension look quite new and eye-catching that's why I suggested it. Thank you for reviewing this issue.
In the screenshot you've sent everything is dark. The proposal of the issue is making only the code blocks dark and leaving the rest light, right?
yes, I proposed to make only code blocks dark not all websites. Just as the above screenshot I have sent (second one).
Sorry, I've misinterpreted the (second) screenshot you've sent. I think this issue is a matter of taste :) Some sites have it one way, others have it the other way; some prefer it one way others prefer it the other way.
Have you seen that applicants can only work on 2, max 3 simple issues (I'm referring to this)? I think you already have 2 simple issues (or more) to work on, don't you? Or do you rely on this one to have something simple? :)
I have given only one simple issue to work on. I would love to contribute more.
Hi @shreyaswikriti,
I agree with @pitag-ha that this is definitely one of those subjective issues, but I think to add a dark colour format now would entail two important aspects:
What do you think?
The ability to toggle between the dark and light?
Yes, I agree that that might be an improvement, if it is for the whole site. @patricoferris, do you think it would also make sense to only have dark mode for code examples? I think that's what the issue is about. @shreyaswikriti , please correct me if I got that wrong and you actually wanted to implement dark mode for the whole site :)
Primarily, I raised this issue to change the color of code blocks only but as mentioned by @patricoferris theme switcher can be added to the whole site. I have seen many websites that do use the theme switcher.
Sounds great. Could you send me a link to a couple of websites with theme switcher? It rings a bell having seen that somewhere, but I don't know where and I'm curious :)
Hii @pitag-ha, there is a theme switcher on the GitHub website itself, where we can toggle between dark and light themes. Actually, theme switcher is famous on that website where users spend a lot of time like book reading websites, coding websites, etc. The reason being dark mode gives less eye strain. For this website, we have to Learn section where users can spend more time reading codes, learning about OCaml, etc. So it would be beneficial for a user to have a theme customize option. This is not a compulsory thing but an additional facility for users.
So, Can I start working on this??
So, Can I start working on this??
Yes! :) Sounds good to me (and I'm pretty sure it also does to @patricoferris ). You're right: I should have made that clearer.
I made a theme switcher for the site. Light mode is the same as previous.Since the video quality is not good, I am attaching some screenshots. Kindly have a look @pitag-ha @patricoferris @gs0510
https://user-images.githubusercontent.com/57635473/114870998-712a5180-9e16-11eb-93fe-d05878ac224d.mp4 Screenshots
@shreyaswikriti This is a great start! I have a few comments:
@gs0510, I have made the changes as suggested. Also, I used the 'WAVE Evaluation Tool ' and it's showing no contrast errors. Kindly have a look.
@shreyaswikriti It definitely looks better! But I think every time you switch to another link, there's a flash where the bright version is loaded first and then the dark version is applied which makes browsing the website uncomfortable. Can you fix that?
The switch can be something like this:
Thanks!
@gs0510,
@gs0510, Kindly Have a look. thanks https://user-images.githubusercontent.com/57635473/115125553-8c4bbb80-9fe6-11eb-9a78-d63df69a4e82.mp4
Looks really good @shreyaswikriti!! you should go ahead and open a PR :)
Programmers often like the dark theme. I found this eye-catching. rather than a light theme, we can convert the editor to a dark theme. I'll start working on this if you like. @patricoferris