Design comparison
Solution retrospective
This one was actually a lot of fun to build, first time i did not have any major issues.
Community feedback
- @OhSorrowPosted about 2 years ago
Hey dear Vitório! Congratulations for completing this challenge! 👏
I suggest you importing mobile background patterns (top and bottom patterns) for displays up to about
900px
wide. Because now the background patterns are not OK for the displays larger than375px
and less than700px
.I hope it helps you to get a better result :)
Marked as helpful0 - @correlucasPosted about 2 years ago
👾Hello Vitório, Congratulations on completing this challenge!
You did a really good work here putting everything together, something you can improve its your code html markup and semantics. You can replace the
<div>
that wraps each card with<article>
you can wrap the paragraph with the quote with the tag<blockquote>
this way you'll wrap each block of element with the best tag in this situation. Pay attention that<div>
is only a block element without meaning.✌️ I hope this helps you and happy coding!
Marked as helpful0@VitorioTuckersPosted about 2 years ago@correlucas Salve Lucas valeu pela moral nao respondo seus comentário mas sempre levo em consideração seu feedback me ajudou muito brigadão pela atenção.
1@correlucasPosted about 2 years ago@VitorioTuckers Tmj man! Achei que vc era gringo pelo sobrenome hahahaha
1
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