Responsive News_signIn using React.js

Community feedback

• 

  T

  Chamu•13,130

  @ChamuMutezvaPosted over 1 year 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

  Marked as helpful

  0

  ---

  

  VIJAY•190

  @VJYGOURPosted over 1 year ago

  @ChamuMutezva thankyou for correcting me , in the next project i will work on these areas and pay attention to these details.

  0