Skip to content
Submitted almost 2 years ago

Responsive news homepage with CSS Grid

Aleksei Kovalev•50
@AlekseiK412
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 think I could do a better job making the webpage responsive on resolutions other than the required 375px and 1440px, and maybe cleaner CSS code. I like the way I implemented the menu bar functionality and the blur overlay though. If there is anything you think I could improve and should know about, please do tell, any feedback is welcome.

Community feedback

  • Adriano•38,390
    @AdrianoEscarabotePosted almost 2 years ago

    Hello Aleksei Kovalev, how are you? I truly loved your project's outcome, however I have some advice that I hope you'll find useful:

    I noticed that you used a button in which case the best option would be an a, because in my head when a person clicks on a button written Read More, he is not confirming a form, or something like, it will be redirected to another page, to Read More about!

    to solve this problem do this:

    <a href="/" class="button">Read More</a>

    images must have alt text unless it is a decorative image, for any decorative image each IMG tag must have empty alt="" and add aria-hidden="true" attributes to make all the assistive technologies of the Web, as screen reader. Learn the differences between decorative/meaningless images vs important content.

    The remainder is excellent.

    I hope it's useful. 👍

    Marked as helpful
    1

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