Skip to content
Submitted over 1 year ago

News Homepage Solution

Kila•30
@Shelton-Rolle
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

Open to any and all feedback! I'm not 100% confident in my responsiveness for this project so I'd love to know how I could improve.

Community feedback

  • Jonathan Kila•290
    @Excelsior2021Posted over 1 year ago

    Hi @Kila,

    You have my surname as your username! 😮

    Well done for completing the challenge.

    Just a few things I noticed:

    1. The font is not being applied to all of the text in the preview site
    2. For both the mobile and desktop views, try to add some margin/padding at the bottom of the page for UI.
    3. The mobile menu, when you scroll down it the menu goes with it, maybe you can try to implement a way for it not to scroll at all when the menu is open?
    4. At a certain breakpoint for the desktop view, the main image no longer lines up with the rest of the content. Something else to fix?

    Apart from that, good job!

    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