@DylandeBruijn
Posted
@Divya4879
Hiya! š
Congratulations on your solution, it looks very close to the design! I can tell you put a lot of effort into it. Nice addition to add your own color-theme! Very good that you completed the project within your estimated time-frame.
Things I like about your solution š
- Responsive
- Clear descriptive CSS classes
Things you could improve āļø
-
Try putting your buttons in a list and changing them into links, this way you get the cursor pointer automatically.
-
Try using a heading tag for the name on the card to make it more semantically correct.
-
Good job on centering your card using
position: absolute
, maybe next time you can use something like Flexbox to center it? -
Try using relative units
rem
andem
for yourfont-sizes
. -
Try using CSS variables to make your values more reusable across your code.
I hope you find my feedback valuable, and I would appreciate it greatly if you could mark my comment as helpful if it was! š
Let me know if you have more questions and I'll do my best to answer them. šāāļø
Happy coding! š
Marked as helpful
@Divya4879
Posted
@DylandeBruijn Thank you so much for your valuable feedback. It's well-thought of and really helpful.
I'll implement the changes soon. Thanks again.