product-preview-card-component-main using - CSS => Grid
Design comparison
Community feedback
- @Nadine-GreenPosted almost 2 years ago
HEY MURAT!
I love the outcome of your solution, however, I would like to give some pointers on how it could be improved.
-
You seem to have forgotten to capitalize the word 'Perfume', you will need to use a
text-trasform
for this. -
You also need to give the button a hover effect, you will need to use a darker green.
-
As for the accessibility warnings, you will need to use a
main
instead of adiv
for the<div class="container">
, this should remove the warning.
HOPE I COULD BE OF HELP :)
HAPPY CODING!
Marked as helpful0@Murat-cmdPosted almost 2 years ago@Nadine-Green It was my first project, I was not even sure that it will come out like this...But still thanks for pointing out the problems, I will sure be careful next time.
0 -
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord