dewslyse• 3,025
@dewslyse
Posted
Hello Chris! Congrats on your submission. Your component looks very good. Some things to note:
- Always remember to include landmarks (e.g.;
header, main, footeretc) in your document. In this case, you could replace the.card-container<div>with a<main>. Also, the.attribution<div>can be a<footer>. - Replace the
background-imageonbodywithpattern-background-mobile.svgat@media (max-width: 435px)
Happy coding!
1
Chris• 250
@chintriago
Posted
@dewslyse Thanks I appreciate the feedback!
0
