Job Listings App with React

Community feedback

• 

  Alex•660

  @al3xback

  Posted over 3 years ago

  Hi Natasya,

  My feedback is:

  • It would be better if we set image on header to be a background-image instead of img tag. The reason is:

  a) this is just static image and never changing too often (even no need for CMS setup for future).
  b) we are not going to use picture element for better rendering.
  c) less code.. haha
  

  • on main, changes will be:

  main {
      ...
      flex-wrap: wrap; //remove
      align-items: center; //remove
  }
  

  we dont need align-items: center cos each .list element has width property 100%, then this has no effect :) and we dont need flex-wrap: wrap cos we dont set exact height to it.

  01

  Marked as helpful
• 

  

  Chamu•13,860

  @ChamuMutezva

  Posted over 3 years ago

  Great solution Natasya. Well done. Filter working as expected plus the site is responsive

  01

  Marked as helpful