Skip to content
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

Choose which Github Dark Theme #844

Closed
2br-2b opened this issue Jan 29, 2021 · 18 comments
Closed

Choose which Github Dark Theme #844

2br-2b opened this issue Jan 29, 2021 · 18 comments

Comments

@2br-2b
Copy link

2br-2b commented Jan 29, 2021

Since there's already 5 PRs about this (one of which is mine), there should probably be an issue for this.

Which dark mode theme should be added?

Our current options:

#691 by @ErickSimoes

ErickSimoes

#758 by @issy

issy

#811 by @microhod

microhod

#823 by @hd4y2t

hd4y2t

#843 by @2br-2b

2br-2b

Related to issues #25, #79, #689, and many others

@ErickSimoes
Copy link

Can I vote for mine?

@ErickSimoes
Copy link

If I can't vote for mine, I choose #758 by @issy

@hd4y2t
Copy link

hd4y2t commented Jan 30, 2021

it's so hard

@2br-2b
Copy link
Author

2br-2b commented Jan 30, 2021

When making my dark theme, I did my best to mirror the colors from a pinned repo from the Github profile screen. I'd post a screenshot, but my screenshot tool is changing the colors in the screenshot for some reason.

@microhod
Copy link

microhod commented Feb 1, 2021

If I can't vote for mine, I choose #758 by @issy

Yeah I agree, if I ignore my preference for my own, I'd vote #758 (although I'd prefer if it was modified slightly so that all the shades of blue match).

In general I feel like the colours 'pop' better in the ones that include the blue.

@alik604
Copy link

alik604 commented Feb 7, 2021

if my vote matters:
id say #758 by @issy. maybe with the offwhite text more white/bright

@2br-2b
Copy link
Author

2br-2b commented Feb 7, 2021

Hi @alik604 ! Of course every opinion matters!

It seems quite clear that #758 is the most popular choice.

@microhod , could you post an example of what you're talking about so that we can compare it with the current version?

Also, could someone who has a working screenshot tool post a picture of #758 in a home screen? That way, we can see what the final result looks like.

@Karthik-Nayak98
Copy link

I would choose #758 by @issy. Because it looks much bright than others.

@2br-2b
Copy link
Author

2br-2b commented Feb 13, 2021

Keep in mind that not everyone uses dark mode. The chosen theme should look good in dark mode and light mode.

@issy
Copy link
Contributor

issy commented Feb 13, 2021

Here is a picture of my theme in action. First pic is in light mode
IMG_20210213_202540.jpg
IMG_20210213_202733.jpg
You can check this out yourself on my profile :)

@Karthik-Nayak98
Copy link

If there is border then we can differentiate between the colors of the card and background.

@issy
Copy link
Contributor

issy commented Feb 14, 2021

If there is border then we can differentiate between the colors of the card and background.

They are the same colour, there is no border

@2br-2b
Copy link
Author

2br-2b commented Feb 28, 2021

If there are no objections, I think it's okay to merge #758 and close the other 4 PRs @anuraghazra

@ErickSimoes
Copy link

I agree 👍

@stale
Copy link

stale bot commented Apr 1, 2021

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale Issue is marked as stale. label Apr 1, 2021
@2br-2b
Copy link
Author

2br-2b commented Apr 3, 2021

Any updates on this?

@stale stale bot removed the stale Issue is marked as stale. label Apr 3, 2021
@anuraghazra
Copy link
Owner

Choosing #758 because of it's good contrast and visibility compared to others.

@anuraghazra
Copy link
Owner

Closed via #758

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

9 participants