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

Responsive news homepage.

@kdrai007

Desktop design screenshot for the News homepage coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
2junior
View challenge

Design comparison


SolutionDesign

Solution retrospective


please give your suggestions.

Community feedback

@Genildocs

Posted

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

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