-
-
Notifications
You must be signed in to change notification settings - Fork 35.9k
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
fix(ui): display checkmark on completed C# challenges #52504
fix(ui): display checkmark on completed C# challenges #52504
Conversation
👀 Review this PR in a CodeSee Review Map |
b9c95f7
to
83f5cdd
Compare
Hey @huyenltnguyen, this is looking good 👍 I was working on this one that does some similar things. I think we should be fine to get this in and I can fix up any conflicts on my PR if they arise. |
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.
LGTM 🎉
Note: I believe this layout is used in the some of the C#, College Algebra, Odin Project, and English Challenges.
I resolved the minor conflict on this cause it would be nice to get it in. |
…2504) Co-authored-by: Tom <20648924+moT01@users.noreply.github.com>
Checklist:
main
branch of freeCodeCamp.I found that completed C# challenges don't have a checkmark next to their title. I'm addressing this by replacing the
h2
element with theChallengeTitle
component, which has the checkmark built-in.Side effects of this change are:
h1
instead of anh2
. This helps make the pages more accessible as C# challenge pages don't have anh1
at the momentScreenshots