💕Animated News homepage Bootstrap💕

Community feedback

• 

  バレンタイン 😈•64,190

  @VCaramesPosted about 1 year ago

  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-HishamPosted about 1 year ago

  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
• 

  Mascud Abdiwahab•170

  @mesutabdiwahabPosted about 1 year ago

  liked it, mate.

  1
• 

  mihapehlic•70

  @mihapehlicPosted about 1 year ago

  Teach me Sensei!

  1
• 

  CodeBot•200

  @Emperor-ShadowPosted about 1 year ago

  Amazing as always

  1
• 

  Amirhosein Fallahan•240

  @PartialTrussPosted about 1 year ago

  fantastic😍😍😍😍😍😍

  1

  ---

  

  Saad Hisham•1,750

  @Saad-HishamPosted about 1 year ago

  Thank you 🥰@PartialTruss

  0
• 

  Paul Ogbonna•100

  @AcerNg548Posted about 1 year ago

  Love the animation

  What package did you use please?

  1

  ---

  

  Saad Hisham•1,750

  @Saad-HishamPosted about 1 year ago

  Animate.css@AcerNg548

  0
• 

  saddamjon•610

  @saddamjon1Posted about 1 year ago

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

  0