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

Dark mode #7176

Closed
paulmelnikow opened this issue Oct 19, 2021 · 17 comments
Closed

Dark mode #7176

paulmelnikow opened this issue Oct 19, 2021 · 17 comments
Labels
frontend The React app and the infrastructure that supports it on-hold Deferred in favor of another approach, blocked on preceding efforts, stale, or abandoned

Comments

@paulmelnikow
Copy link
Member

As requested in #7129, it would be helpful to add a dark mode design to Shields.

Initially this probably can use the system light / dark setting.

In terms of process, I'd suggest creating a mockup or a draft PR, doing a quick design review, and then getting approval on the design. We can handle the technical aspects of the code afterward.

@paulmelnikow paulmelnikow added frontend The React app and the infrastructure that supports it good first issue New contributors, join in! labels Oct 19, 2021
@Andre601
Copy link

Personally, I think using a lighter or darker variant of the badge's grey (#555 iirc?) would be an interesting idea.
It would fit shield's main focus, which is badges by making the colour go alongside that of the badges main one.

@tapanchudasama
Copy link
Contributor

This looks interesting. I'd like to pick this up.

@calebcartwright
Copy link
Member

This looks interesting. I'd like to pick this up.

Awesome, please do! You can reach us here and/or on Discord if you have any questions or want to chat about it

@tapanchudasama
Copy link
Contributor

Okay, so I went thought the issues related to this topic of website design. And where do you think we should start from? Like how should i start work on this?

@paulmelnikow
Copy link
Member Author

Hit me up on Discord and let's find a time to chat about it! I would also suggest that we start working together on something that is focused and small (like dark mode!), and build up from there to bigger and bigger things.

@tapanchudasama
Copy link
Contributor

Yep, will do. How can i reach you on discord?

@chris48s
Copy link
Member

@Harsh-Shedge
Copy link

Hi, Can I work on this issue?

@calebcartwright
Copy link
Member

calebcartwright commented Jul 3, 2022

Hi, Can I work on this issue?

Thank you for the offer! I don't think there would be much point in trying to implement new features such a large new feature on top of the existing UI given the plans to overhaul it (see #7815). If you'd like to help with frontend work though, there's probably some other items discussed in that issue which may be up for grabs

@PaulaBarszcz
Copy link
Collaborator

Hi @calebcartwright,
Since this issue should not be picked up before #7815 is done,
could it be closed?
To avoid more time spent by people on analyzing it :)
Thanks

@Andre601
Copy link

I feel like closing this issue would only give the wrong impression. That of it being completed/done while it isn't.

It's understandable that people randomly appearing here and trying to work on this while it shouldn't is somewhat annoying, but closing the issue isn't really the best solution here due to the aforementioned reasons.

A solution could perhaps be a pinned issue (Perhaps #7815 itself?) to give a more prominent indicator that something is currently at work.

@calebcartwright calebcartwright added on-hold Deferred in favor of another approach, blocked on preceding efforts, stale, or abandoned and removed good first issue New contributors, join in! labels Jul 20, 2022
@calebcartwright
Copy link
Member

Since this issue should not be picked up before #7815 is done,
could it be closed?

@PaulaBarszcz - I agree with the thrust of what @Andre601 said against closing, though I've adjusted the labels on both issues instead of pinning as that's more inline with the way we represent status/dependencies.

@PaulaBarszcz
Copy link
Collaborator

In #7815 even the amount of comments is intimidating, thus I will search for something with more solidified criteria to pick up:)

Thank you both for your answers, and for labels adjustment. Have a great day 🍀

@calebcartwright
Copy link
Member

@PaulaBarszcz - are you looking for general opportunities to contribute to Shields, or were you specifically interested in the front end pieces? (and if either is true, that would be great! 🙏 🎉).

If it's the latter, it's indeed fair to say there's a large volume of up front analysis and then work needed before smaller, more granular pieces could be approach. However, if it's the former there's plenty of smaller, more solidified items we'd be happy to recommend!

@PaulaBarszcz
Copy link
Collaborator

Hi @calebcartwright,
I just began my journey with open source projects and I am searching for projects to which I would be able to contribute. For now I feel that I would like to keep to small-scale tasks to gain confidence and fluency in the open source environment :)

That said, it would be great if you could recommend some tasks good for a Regular Front-End Dev starting her adventure with Open Source ☺️
Thanks for asking 😉

@calebcartwright
Copy link
Member

Hi @calebcartwright, I just began my journey with open source projects and I am searching for projects to which I would be able to contribute. For now I feel that I would like to keep to small-scale tasks to gain confidence and fluency in the open source environment :)

That said, it would be great if you could recommend some tasks good for a Regular Front-End Dev starting her adventure with Open Source relaxed Thanks for asking wink

Unfortunately I think any and all of our pure front end work (React and TypeScript) is going to similarly be blocked.

If you're willing to give server-side JavaScript a try then I think we've got a few open issues that would be viable candidates. I want to be sure we don't take this particular issue off topic, but if you're interested then feel free to drop us a note in Discord (https://discord.gg/HjJCwm5) or just use an emoji reaction to this comment (let's say the Rocket emoji 🚀) and we'll tag you on a couple such issues for consideration. No worries if not!

@chris48s
Copy link
Member

We have just launched a completely redesigned frontend https://shields.io/ which includes light/dark mode themes + toggle

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
frontend The React app and the infrastructure that supports it on-hold Deferred in favor of another approach, blocked on preceding efforts, stale, or abandoned
Projects
None yet
Development

No branches or pull requests

8 participants
@paulmelnikow @chris48s @Andre601 @calebcartwright @PaulaBarszcz @tapanchudasama @Harsh-Shedge and others