Responsive News_signIn using React.js

Community feedback

• 

  

  Chamu•13,860

  @ChamuMutezva

  Posted almost 2 years ago

  Hi VIJAY

  So far so good with your project. Here are some areas that needs addressing:

  • alt img elements should have an alt value. Decorative images should have an empty value , if not decorative, then the value should describe the message in the image
  • landmark elements such as main should be part of your HTML structure . Other landmark elements include the header and footer elements. You can get a detailed explanations about these elements from MDN
  • when you are presented with various images , where you are a given an image to display for mobile, and another for desktop etc - they are recommended practices that should be used with the image element. One of them is the picture element. Using JS is not a recommended practice even though it may work.
  • the input element is associated with a label element whereby the label should have an htmlFor attribute whose value is the id of the input element.
  • css styling should follow a mobile first approach

  01

  Marked as helpful