Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Blogr landing page

#accessibility#jss#sass/scss#vanilla-extract#materialize-css

@ovie-best

Desktop design screenshot for the Blogr landing page coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
2junior
View challenge

Design comparison


SolutionDesign

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

Community feedback

Md5 dalton 1,430

@md5dalton

Posted

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

0

@ovie-best

Posted

@md5dalton Thanks a lot. I will try it out.

0

Please log in to post a comment

Log in with GitHub
Discord logo

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