sixth project on Frontend Mentor

Community feedback

• 

  Agnik Bakshi•480

  @Agnik7Posted over 1 year ago

  Hi,

  Congratulations on completing this challenge!!🎉🎉🎉🎉

  I have some suggestions that might help you.

  • For the body, you don't need to define width:100% and align-items: flex-start as these as default values. Even if you don't mention it, flex-start and width:100% is applied by default. You can use align-items to center the elements of the body. To learn how to center a div, click here.

  • Don't make the height of the body element constant by defining the height. Instead, define the min-height to make the body responsive. min-height: 100vh.

  • For the container, define the max-width instead of width for better responsiveness.

  • Provide a left and right margin of 2rem to the description. Although your layout is perfect, your margin and padding are off.

  • Try using relative units like rem and em more than absolute units like px, to aid in better responsiveness.

  • Provide a meaningful alt text because it is what is going to be displayed if the image cannot be displayed. So, giving a meaningful alt lets the user know what the image is about.

  Hope this feedback helps you in your coding journey. Have a nice day!!

  Marked as helpful

  1