-
Notifications
You must be signed in to change notification settings - Fork 69
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
Redesign the lesson content page #2321
Comments
Is anyone working in this issue? I want to work on this issue. Can @flacial explain a bit more about it? |
No one is working on this issue. The main thing to do is to change the page's design to this mock design. Doing so will fix some UI issues that affect the student experience while studying (also polish the page). If you're planning to take this issue, let's chat on Discord. My username there is |
@flacial It was my bad.....in understanding your issue. Unfortunately i am not a designer. I thought you have the design and You need me to code this up. Anyways.....I am new here, exploring open source issues. Nice talking to you. |
From what I understand, the designs are ready and someone needs to code them up. I think you misunderstood @flacial's words |
After a bit of tweaking, thanks to @flacial for the help, here is the new lessons page redesign to match our new design, there are two variants, left TOC and right TOC, we can vote on which one we should implement. If you guys have any feedback, i would love to hear it. |
@SlyBouhafs I vote for Left TOC page. It looks perfect! |
I agree, that header is mostly for students/mentors, we need to adapt it so that it works with the admin pages too, i will work on that too. |
Things to redesign
Lesson parts nav
Large and unstyledElements spacing & letter spacing
Margin between elements and codeblocksFonts
The fonts are inconsistent. The should be `Inter` fontChallengebar font, size, colors, weight, and letters spacing
Table of content
It's too largeText length
It's hard to read lengthy textWe can use the design mock we created in the past for this page. I think it would do the job.
The text was updated successfully, but these errors were encountered: