Hello, Caique Carvalho da Silva! 👋
Good job on this one! 🎉 Your solution responds well and overall looks good. Here's my suggestions:
- The
div
element has no special meaning at all. Use of more appropriate elements instead of the div element leads to better accessibility for readers and easier maintainability for authors. For.header-title
use heading<h1> - <h6>
element. For.header-subtitle
you can use paragraph<p>
element. This also applies to text in other sections. - On mobile resolution add
width: 100%
to.main-container
and removemax-width: 400px
so your container will stretch across different screen sizes. Keepmax-width: 660px
on 660px media query to prevent it for stretching too much. - Use unordered list
<ul>
for.reasons-list
. HTML lists are used to present list of information in well formed and semantic way.
Good luck with that, have fun coding! 💪
Marked as helpful
1