Responsive news homepage.

Community feedback

• 

  P

  Genildo Cerqueira Souza•470

  @GenildocsPosted over 1 year ago

  Hi there! Congratulations on completing the challenge!

  As a suggestion, I believe you could have used display grid to define the entire layout of the page and use flex-box only on one-dimensional elements. For example, by applying grid to the "main" block, you could align the elements more efficiently.

  For the main image, I recommend not defining a fixed height and instead letting the element take up 100% of its parent element or using the max-height property for better responsiveness to different screen sizes.

  Lastly, I suggest using grid in the "bottom" section and flex-box on child elements for a better outcome.

  I hope these suggestions are helpful in improving your work!

  0