Skip to content
OverviewSolutions
8
Comments
3
Hilverton Cesar
@Hilverton

All comments

  • Hilverton Cesar•135
    @HilvertonPosted almost 2 years ago
    • To fix the html error I suggest replacing the section tag with a div, as it has no text content inside.
    • To center the main content, you could add the following code:
    body {
  display: flex;
  align-items: center;
  justify-content: center;
  height: 100vh;
}

    and strip the y-axis margin of the <main class="container"> element.

    Happy coding

    0
  • Hilverton Cesar•135
    @HilvertonPosted over 2 years ago

    Sua implementação ficou muito boa, gostei muito das animações, eu só daria pequenas dicas:

    1. Você poderia centralizar o ícone no botão de busca de conselhos
    2. Aumentar o espaçamento entre a imagem da divisão e o botão com o ícone
    3. Colocar os ícones em uma pasta "assets" dentro da pasta "src", é mais profissional

    De resto, uma ótima implementação!

    Marked as helpful
    1
  • Patrick VanVorce•125
    @pcloverdevelopmentSubmitted over 4 years ago

    Hey everyone! I went back and took the time to understand how to use flexbox for positioning elements and reworked this solution.

    I decided to make my breakpoint at 450px, I just felt that is what looked best in terms of RWD.

    Looking for possible feedback on how I used padding. Questions I have is where it is better for me to use margins, was there a simpler way for me to do my layout in terms of padding/margins, and could I have used relative padding rather than fixed.

    Happy coding!

    Responsive page using Flexbox!

    2
    Hilverton Cesar•135
    @HilvertonPosted over 4 years ago

    Perfect! Congratulations!

    1