First time ever using React

Community feedback

• 

  P

  Marko Nikolajević•1,065

  @MarkoNikolajevicPosted over 4 years ago

  As your first time using React you did it really really damn good! Your code is clear and readable! You create a JobBoard component and you render all jobs looping with this component. I always use sass with React. I don't have else to say except that you really did a great work with this.

  2

  ---

  

  Diarrah•3,418

  @DiarrahPosted over 4 years ago

  @MarkoNikolajevic wow thanks so much for the kind words!

  0
• 

  Roman Filenko•3,335

  @rfilenkoPosted over 4 years ago

  Hey, it's ok to use sass with react, there some different solutions like css-in-js (popular styled components, for example), but those are more advanced and overkill for this project. A few tips for you:

  • don't mutate state directly in react (saw push method in your code);
  • probably don't make dots as li items, make them as pseudo elements;
  • maybe add some transitions to selecting/ deleting tags. Also layout moves, when .job-requirements div appears.

  Other challenge for you would be to make dynamic jobdetails pages with addition of router between pages 😉.

  Roman

  0

  ---

  

  Diarrah•3,418

  @DiarrahPosted over 4 years ago

  @rfilenko thanks so much! I will try to revise the code to take that push method out.

  0