-
Notifications
You must be signed in to change notification settings - Fork 1
update(ui): redesign lesson card component #1004
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
base: master
Are you sure you want to change the base?
Conversation
51b1e33
to
fc60c8c
Compare
fc60c8c
to
c1aa29f
Compare
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.
c1aa29f
to
b7205c8
Compare
b7205c8
to
3802386
Compare
topics: range(7).map(() => faker.lorem.words(1)), | ||
level: "C1", |
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.
There is no English level for tutors. This should only appear in students cards for tutors.
Furthermore, lesson-card for students is different from lesson-card for tutors.
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.
You better write this in LessonCardV2 for instance.
<Typography | ||
key="remaining" | ||
tag="span" | ||
className="border border-natural-500 rounded-[200px] px-1.5 py-1" |
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.
specify the text color, size, and width to match the prototype.
Quality Checklist
Links
Important
Incase you have any valuable knowledge that you think that it is worth writing down, feel free to add it to our handbook. Wounder why we are doing this? check this guide.