Skip to content
Submitted over 1 year ago

News homepage

SilentCoder•20
@Jamiu10012
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

🗞️ Hello everyone,

I am excited to share my solution for the News Homepage Challenge!

I welcome any feedback on my solution and am always looking to improve my skills and make my code cleaner. Thank you for taking the time to check out my work!

Community feedback

  • Hussain Hussain•410
    @LysitheaDarkKnightPosted over 1 year ago

    Hello, how are you? Good job on completing the challenge!

    It's usually unnecessary to define width: 100% for elements and it can even cause you problems later on. The reason being is when you apply a margin to it, it'll overflow outside the element and cause you problems in case you want to add more to your element. Usually, width: auto (it's the default value) or defining it manually by does the job.

    Happy coding!

    Marked as helpful
    0
    SilentCoder•20
    @Jamiu10012Posted over 1 year ago

    @LysitheaDarkKnight thanks for the feed back

    1
  • hanny•170
    @hannyfishPosted over 1 year ago

    Just wanted to hop in to say for a first submission that is utterly stellar! Looking forward to seeing what else you come up with!

    0
    SilentCoder•20
    @Jamiu10012Posted over 1 year ago

    @hannyfish thanks

    0
    SilentCoder•20
    @Jamiu10012Posted over 1 year ago

    @hannyfish thanks

    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