-
-
Notifications
You must be signed in to change notification settings - Fork 39.1k
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
feat(learn): create a progress indicator map #53963
feat(learn): create a progress indicator map #53963
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for working on this @a2937,
We should centre the numbers inside the certifications.
I am not sure if I like the arrows, it might me useful to try a couple of them.
Also, arrows should be white in the dark-theme and black in the light-theme.

The arrows aren't exactly a character. They're an SVG as it was the only way I could make something grow and shrink on the page. |
Let me know if you need any help @a2937! |
I actually might need a little help making a new arrow shape, if that's alright. Though I'm fairly sure the colors are adjusted for night mode. Additionally, while centering isn't my strong suit; I hope it's closer this time. |
Hey @a2937, Sorry for keeping you waiting so long, I will check it today. |
Thank you Sembauke. Those arrows look very lovely. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmmm...what kind of indication should I give that says the camper has claimed the certification? |
I would hide the ribbon part on unclaimed certs, but leave the circle dark. |
I can't hit the "I agree to freeCodeCamp's Academic Honesty Policy." button so I can claim my certifications. But I will send in my update. I'm also not sure if I added the logic right. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@a2937, thank you for the hard work here. |
No problem. I'll look into it and start that follow up PR soon. |
Checklist:
main
branch of freeCodeCamp.Closes #50412
It's a revival of this PR. #52686