Skip to content
Submitted over 2 years ago

Blogr landing page

accessibility, jss, sass/scss, vanilla-extract, materialize-css
Ovie Nathaniel•220
@ovie-best
A solution to the Blogr landing page challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

Hello Frontend Mentor !!! This is my solution to the Blogr Landing Page Challenge.

I was able to practice some of my CSS layout skills, and some things were difficult, but I eventually found my way out.

You can also take a look at my code through the repository link. The code is not clean though.

I am still learning

Code

Please log in to post a comment

Log in

Community feedback

  • Md5 dalton•1,430
    @md5dalton

    Hi Oviedev. Good work on your solution.

    Here are a few of my suggestions:

    • Remember to add alt attribute to your img tags for accessibility purposes.
    • In your Javascript code I've noticed that document.getElementById has been repeated quite a couple of times, so to keep your code DRY (Do not Repeat Yourself), why not write a helper function to handle that, so your code becomes a little bit leaner and easy to maintain:
    const getElement = selector => document.querySelector(selector)

    You can even go further by replacing showItems() and hideItems() with toggleItems().

    Please feel free to ask if you need further explanation.

    Marked as helpful

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