Travolgi 🍕• 31,420
@denielden
Posted
Hi Giorgi, congratulations on completing the challenge, great job! 😁
Some little tips for optimizing your code:
- you can use
articletag instead of a simpledivto the container card for improve the Accessibility - Using
<hr>for the line is not the best way because this tag have a semantic meaning... in this case use div withborder-bottombecause this line is decorative - add
transitionon the element with hover effect - instead of using
pxuse relative units of measurement likerem-> read here
Hope this help! Happy coding 😉
Marked as helpful
0
Giorgi• 200
@siduki
Posted
@denielden Hello.
Thank you for your feedback, I will definitely consider it in the future.
Relative units are little confusing for me yet, I think it will come with practice.
1
Travolgi 🍕• 31,420
@denielden
Posted
@siduki You are welcome and keep it up :)
0
