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

Crowdfunding Product Page Main

@jvssvj

Desktop design screenshot for the Crowdfunding product page coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
2junior
View challenge

Design comparison


SolutionDesign

Solution retrospective


Hello everyone, my name is João Vitor and I completed another frontend mentor challenge.🔗

How can i improve and reduce unnecessary code? Leave your comment. 🙂

Community feedback

@HusseinSamy

Posted

Hello @jvssvj, 👋🏻

Awesome job on taking the first steps towards your dream!

I'm thrilled to see your engagement with this platform to enhance your design skills. I like your code organization. Though, I have some additions to make more clean:

1. Wrap all of your code inside function and call it when needed:

====> Instead of writing it like this
openMenuBtn.addEventListener('click', openCloseMenu)
closeMenuBtn.addEventListener('click', openCloseMenu)


====> Write it like this 
function addEventListnersForMenuButtons(){
   openMenuBtn.addEventListener('click', openCloseMenu)
   closeMenuBtn.addEventListener('click', openCloseMenu)
}

====> Call it in the right place
addEventListnersForMenuButtons()

2. Use Bem for naming your styles.

3. Add meta tags to your project:

<meta name="title" content="My Website">
<meta name="description" content="My Website Description">
<meta name="keywords" content="Some, Keywords, Here">

Hope you find it useful!😇

Keep up the fantastic work, and happy coding! 🚀

Marked as helpful

1

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