Skip to content
Submitted almost 2 years ago

Responsive News Page

Cj Rivas•50
@Cjlikescode
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...News homepage
  • HTML
  • CSS
  • JS
2junior

Design comparison

SolutionDesign

Solution retrospective

I found the hamburger meu to be challenging, I still did not get the transparent background when the menu was open but I'm sure I overlooked it. I feel like I need to work on containers and how to center everything correctly. But I think I did decent on my first project. Any feed back is welcome.

Community feedback

  • Travolgi 🍕•31,400
    @denieldenPosted almost 2 years ago

    Hello Cj, You have done a good work! 😁

    Some little tips to improve your code:

    • you can use article tag instead of a simple div to the container of news to improve the Accessibility
    • remove all unnecessary code, the less you write the better as well as being clearer: for example the div container of image

    Keep learning how to code with your amazing solutions to challenges.

    Hope this help 😉 and Happy coding!

    Marked as helpful
    1
    Cj Rivas•50
    @CjlikescodePosted almost 2 years ago

    @denielden thank you for the feedback very appreciated will take those notes into account!

    1
    Travolgi 🍕•31,400
    @denieldenPosted almost 2 years ago

    @Cjlikescode You are welcome and keep it up :)

    0

Please log in to post a comment

Log in with GitHub

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