A simple challenge from Frontend Mentor solved using HTML, CSS and JS

Community feedback

• 

  P

  tediko•6,680

  @tedikoPosted over 3 years ago

  Hello, Vishal! 👋

  Well done on this challenge! Responds well, and looks good. What I can advise is:

  • Since your .header logo image is decorative your alt text should be provided empty (alt="") so that they can be ignored by assistive technologies, such as screen readers.
  • Use <button> element instead of <input> with type of button. Both causes the form data to be submitted to the server but The difference is that <button> can have content, whereas <input> cannot (it is a null element). While the button-text of an <input> can be specified, you cannot add markup to the text or insert a picture. So <button> has a wider array of display options.
  • Wrap your social-icons into <a> anchor tags, since they'll redirect to another page.

  Good luck with that, have fun coding! 💪

  0

  ---

  

  Vishal•330

  @vshal-chPosted over 3 years ago

  @tediko Thank you very much for the feedback😁,

  Happy coding😁.

  0