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

Submitted

Used flexbox for most part of the layout

Neo• 40

@Neo-Mengwai


Design comparison


SolutionDesign

Solution retrospective


Any feedback is welcome

Community feedback

Adarsh Pratap• 5,515

@adarshcodes

Posted

Hi @Neo-Mengwai, Congratulations on your first challenge and welcome to this amazing community. You did amazing work on this challenge, let's see your solution from different aspects and Please generate a new Screenshot it will be more useful to compare your design.

  1. Design aspect
  • Your design looks fine but need some improvements.
  • Firstly remove the border of your .container class.
  • Remove margin from the top, it leaves a white little space.
  • Increase the size of the image.
  • And you can change the color of your attribution links so that it would be easily noticeable.
  1. Responsive aspect
  • You can add more breakpoints for better responsiveness.
  1. Code aspect
  • The code looks clean.
  1. Issues aroused
  • You can also solve those HTML issues.

Keep coding

1

Neo• 40

@Neo-Mengwai

Posted

Thank you very much.

0

Please log in to post a comment

Log in with GitHub
Discord logo

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