I will appreciate your feedback as well as your advices to have a better (shorter ?) code.😀
steezyza
@steezyzaAll comments
- @pyko7Submitted almost 3 years ago@steezyzaPosted almost 3 years ago
This is a good attempt, well done.
The only issues I noticed
a) accessebility: Semantic HTML helps code readers navigate through the page easier e.g.
<header> <main> <footer>
and also includealt
in your image tag e.g.<img src="./assets/images/image-avatar.png" alt="nft-card">
b) the image should have a border radius e.g.
border-radius: 5px;
All the best for future solutions
Marked as helpful0 - @skippysworldSubmitted almost 3 years ago
Actually, I'm not sure how to specify my questions since it's my first challenge, but would like to get an overall feedback on css, whether it's not over-complicated. Might it be done in easier manner?
Thanks in advance everyone who finds a minute :)
@steezyzaPosted almost 3 years agoClean code and great solution! @skippysworld
Marked as helpful1 - @CMconsultsSubmitted almost 3 years ago
I would love to learn a better way to do media queries that will be responsive on all devices.