Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

All comments

  • @Fazyyy

    Posted

    Good effort but I think you overlook a few of the smaller details here. In the design the right hand image takes up exactly half of the width, whereas in your design it's more like a third or so? Additionally the spacing between some of your elements is a bit off. If you have an image editing tool it can be handy to check the distances between elements. For example, the gap between the statistics and the paragraph above them.

    0
  • Devesh 120

    @devesht21

    Submitted

    First try! How is it? What are the mistakes that I should not repeat next time?

    @Fazyyy

    Posted

    Not really a criticism but just an option for next time, your parent section with the class "section-qr" seems to only provide the functionality of making the background color full screen. Next time maybe consider applying the background color to the body element and giving it the css attribute of min-height:100vh;

    As others ahve pointed out, using a "main" element rather than a "section" would have been more appropriate.

    Marked as helpful

    0