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

news-homepage-main

@TaofeeTech

Desktop design screenshot for the News homepage coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
2junior
View challenge

Design comparison


SolutionDesign

Solution retrospective


i really don not know how to change the menu svg to the menu close svg

Community feedback

Adriano 34,090

@AdrianoEscarabote

Posted

Hello Taofeeq Olatigbe, how are you? Welcome to the community of front-end mentors! 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="/">Read more</a>

I noticed that in higher resolutions the content is stretching a lot and this is harming the design of the project, to fix this in a simple way we can use a max-width.

nav,
.container,
.blog {
    max-width: 1440px;
}

The remainder is excellent.

I hope it's useful. 👍

0
Sarib 20

@Sarib-Siddique

Posted

Hello, Because you have to learn javascript to change menu SVG to closing SVG. By the way great work, You did well.

0

@TaofeeTech

Posted

@Sarib-Siddique thank you so much

0

@SinisaVukmirovic

Posted

Hello!

You need JavaScript code to change the menu icon SVG.

Also, your "VIEW CODE" link leads to an empty repository.

0

@TaofeeTech

Posted

@SinisaVukmirovic oh okay thanks,

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