Mobile-first using Flex

Community feedback

• 

  Tomiwa Orimoloye•1,000

  @tomiwaorimoloyePosted over 3 years ago

  Everything looks good but you should set the border radius so the site looks more identical with the design.

  Marked as helpful

  0

  ---

  

  Jan•135

  @trostjanPosted over 3 years ago

  @ayoelx forgot about that, thanks for the reminder! :)

  0
• 

  Ayoub1Dev•195

  @Ayoub1devPosted over 3 years ago

  When I previewed your site I found a vertical scroll. Using DevTools I found that you did not reset the margin of the body.

  • Like this:

  body {
     margin: 0;
  }
  

  This will remove the extra space and your wrapper will be 100% of the viewport.

  TIP It is a good practice to start with this on your projects, which will set all elements to the styles inside the curly { } brackets.

  *, *::before, *::after { 
       box-sizing: border-box;
       margin: 0;
       padding: 0 ;
  }
  

  Hope this helps.

  Marked as helpful

  0

  ---

  

  Jan•135

  @trostjanPosted over 3 years ago

  @Ayoub1dev noted! and corrected :)

  0
• 

  SM•30

  @Sarthakm11Posted over 3 years ago

  Hy Jan, Nice work but your "Learn more" buttons are actually not responding to any kind of hover/click, so you can look into that !

  0