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

💕Animated News homepage Bootstrap💕

Saad Hisham 1,750

@Saad-Hisham

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


Hi there🙋‍♂️ i really enjoyed solving this feedback is welcome

Community feedback

@VCarames

Posted

Hey there! 👋 Unfortunately, there are countless things that need to be improved that I would not expect at this challenge level...

For example:

  • This should be a header element and not a div.
  • Company logo alt descriptions should NEVER ❌ be "logo"; it should always state the company's name.

  • Your nav button is inaccessible ⚠️. This because you built it using a div (<div class="open-nav-div">) when it should off been built using a button.

  • The list items inside the nav menu should be wrapped inside an anchor element in order for them to be interactive.

These are just a few things that need to be fixed... there are a whole lot more things that need improvement.

I strongly recommend checking out https://web.dev/learn/html/ to enhance your web development knowledge. This resource can be instrumental in helping you avoid the types of mistakes that may arise in your future projects.

Upon reviewing your profile, I've noticed that you appear to be rapidly tackling various challenges, with near-daily postings. It seems that your primary emphasis is on animation rather than the quality and accessibility of your code.

As a web developer, it's crucial to make quality code (error free, semantically correct, proper usage of elements, etc...) and accessibility your top priorities. These aspects are what distinguish an average developer from an outstanding one.

If you have any questions or need further clarification, feel free to reach out to me.

Happy Coding! 👾

Marked as helpful

4

Saad Hisham 1,750

@Saad-Hisham

Posted

Oh thanks this is my first challenge to use semantic HTML tags I don't really think that they matter since I can customize all their properties with CSS and JS but I will give them more attention in future projects thank you for your time and comment 😀 @vcarames

3

@mihapehlic

Posted

Teach me Sensei!

1
CodeBot 200

@Emperor-Shadow

Posted

Amazing as always

1

@PartialTruss

Posted

fantastic😍😍😍😍😍😍

1

Saad Hisham 1,750

@Saad-Hisham

Posted

Thank you 🥰@PartialTruss

0

@AcerNg548

Posted

Love the animation

What package did you use please?

1

Saad Hisham 1,750

@Saad-Hisham

Posted

Animate.css@AcerNg548

0
saddamjon 610

@saddamjon1

Posted

Hello, Saad Hisham, make it a little lower, the rest is great

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