mobile first, flex-box

Community feedback

• 

  Raymart Pamplona•16,100

  @pikapikamartPosted about 3 years ago

  Hey, awesome work on this one. Layout in desktop looks great just needed for it to bit bigger. The site is responsive and the mobile layout looks great as well.

  CyrusKabir already gave a great suggestions, going to add some as well:

  • The .stat-container selector could use an ul element since those are "list" of information about the company website.
  • Each text-content should be inside a meaningful element like p tag and not just div.
  • Lastly, the .attribution needs to be inside footer so that it is inside a landmark element.

  To be honest the site looks really great and the markup looks great as well. Really great work on this one.

  Marked as helpful

  0
• 

  CyrusKabir•1,885

  @CyrusKabirPosted about 3 years ago

  hello my dear friend ♥ your did good and clean at all but here some problems :

  • you need have more white space in your card content i mean it's not look like main design
  • you need change the white space between the numbers it's a lot
  • you can use filter property or background-blend-mode because you can get closer to design with these things but liner gradient it's just hard you know

  Marked as helpful

  0