Responsive Landing Page Using CSS Flex-Box

Community feedback

• 

  Ozan Zorlu•280

  @zorluozanPosted over 2 years ago

  Hi Annas,

  Nice work. I look at your code and here are some suggestions if you want to improve your project;

  1- About the responsiveness, I think you misunderstand some points. In responsive design, there are two approaches which is mobile-first and desktop-first. In mobile-first approach, you write your code according to the mobile design and use min-width for the media queries to get desktop version. You should make some research about responsive design.

  2- You should check active states of the project. You didn't implement any active state such as when hover them icons and items in footer.

  3- If you look at the design, you should check learn more links. The positions under the learn more links is closer.

  4- You must use alt attributes to all images. I guess you forgot to put some of the alt attributes but that is okey. You can fix that easily.

  5- You should check for the mobile menu position from the design.

  6- Try to use semantic html elements for accessibility such as main,header,footer,section etc.

  That's all I wanted to say about. Happy coding :)

  Marked as helpful

  1