To-do App using ReactJS, SCSS, and localStorage

Community feedback

• 

  Alex•660

  @al3xback

  Posted over 3 years ago

  Hi Natasya,

  Looks like for the header bg is not fully cover the entire screen, esc when viewed on larger device. Try to zoom out the browser and see.

  My feedback is:

  • add full width to .header__bg__image and inheritance to .header__bg__image img selector.

  .header__bg__image {
     ...
     width: 100%;
  }
  
  .header__bg__image img {
     ...
     width: inherit;
  }
  

  • for All, Active, Completed, and Clear Completed would be good if we set it into button element.

  <button type="button">All</button>
  <button type="button">Active</button>
  <button type="button">Completed</button>
  <button type="button">Clear Completed</button>
  

  28

  Marked as helpful