Rock paper scissors lizard spock using React

Community feedback

• 

  

  Matt Studdert•13,611

  @mattstuddert

  Posted about 5 years ago

  Hey Marko, awesome work on this challenge. Your React code looks absolutely fine. I would only make small changes, such as:

  • Instead of using element.style.display = 'none' in various places I would use a logical operator to conditional render the component or not. This would mean the RulesModal in your App.js would become { rules && <RulesModal setRules={setRules} />}. You'd then be calling setRules(false) to close it in the model component.
  • I'd also recommend breaking up your code into smaller components. For example, instead of having 5 button elements repeated in the Game.js you'd have 5 calls to a component that would render a button and change the props based on what's passed down.

  These are small things though. Overall, you've done a really good job. How are you liking React so far?

  21
• 

  siddtheone•490

  @siddtheone

  Posted about 5 years ago

  Very nice, however at resolution ~800px, there is white background with purple background

  00