@huyphan2210
Posted
Hi, @ayx234
I took a look at your solution and have some thoughts:
- Semantic HTML: I think your
.card
could be anarticle
element, or you could make the card itself themain
element instead of usingmain
as a wrapper. Also, the.attribute
section might work better as afooter
element below themain
rather than adiv
inside it. - CSS: You've done a great job styling the card! My recommendation would be to reorganize your CSS file. Consider placing
@font-face
at the beginning and moving all@media
queries to the end. I also noticed that you're styling thebody
in two places, so merging those would improve clarity.
Hope this helps!