Skip to content
Submitted almost 3 years ago

Mobile first workflow with flex components for desktop layout.

Asiimawe•150
@Pablo-cyber21
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Social proof section
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

Hello my fellow frontend lovers , I had some issues when it came to the mobile layout but with a little bit of struggle I was able to get the right alignment with flexbox . I have decided to learn grid to since it seems so easy to create fluid layouts with it, All feedback on this project would me much appreciated.

Community feedback

  • Aakash Verma•6,710
    @skyv26Posted almost 3 years ago

    Hi! Asiimawe, You did it nicely, I find some issues in your work.

    1. Your design text contents needs line-height: 1.5; to make some space between the lines. As you can also see in your design comparison that your text lines are about to collapsing and from the user interface point of view it is little bit unimpressive. So Fix it, when you fix it then see your result will more good as compare to previous one.

    2. Font size is little bit big than expected.

    3. Your main heading paragraph have different color than design preview (Negligible issue)

    4. There are 18 HTML issue and it need to fix ASAP.

    Overall Good. I hope you understand .

    Good Luck

    Marked as helpful
    0

Please log in to post a comment

Log in with GitHub

Join 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