From df60f0c4b457b5b5a39dbce155e4ed451152ea40 Mon Sep 17 00:00:00 2001
From: Angelos Chalaris <chalarangelo@gmail.com>
Date: Thu, 18 Oct 2018 19:11:53 +0300
Subject: [PATCH] Update StarRating.md

---
 snippets/StarRating.md | 18 +++++++-----------
 1 file changed, 7 insertions(+), 11 deletions(-)

diff --git a/snippets/StarRating.md b/snippets/StarRating.md
index 472d60080..72610f894 100644
--- a/snippets/StarRating.md
+++ b/snippets/StarRating.md
@@ -5,9 +5,9 @@ Renders a star rating component.
 Use the value of the `rating` prop to determine if a valid rating is supplied and store it in `state.rating` (or `0` if invalid or not supplied).
 Initialize `state.selection` to `0`.
 Create two methods, `hoverOver` and `setRating`, that take a number as argument and update `state.selected` and `state.rating` according to it, bind them both to the component's context.
-In the `render()` method, define a functional component, called `Star` that will render each individual star with the appropriate appearance, based on the parent component's `state`, and handle its `onMouseEnter` and `onClick` events, using the parent component's `hoverOver` and `setRating` methods.
-Render a `<div>` to wrap the `<Star>` components, which are created using `Array.prototype.map` on an array of 5 elements, created using `Array.from`, and handle the `onMouseLeave` event to set `state.selection` to `0`. 
-Finally, pass the appropriate values to each `<Star>` component (`starId`, `marked`, `onHover` and `setRating`).
+In the `render()` method, define a functional component, called `Star` that will render each individual star with the appropriate appearance, based on the parent component's `state`, and handle its `onMouseEnter` event, using the parent component's `hoverOver` method.
+Render a `<div>` to wrap the `<Star>` components, which are created using `Array.prototype.map` on an array of 5 elements, created using `Array.from`, and handle the `onMouseLeave` event to set `state.selection` to `0` and the `onClick` event to set the `state.rendering` to the `star-id` attribute of the `event.target`. 
+Finally, pass the appropriate values to each `<Star>` component (`starId`, `marked` and `onHover`).
 
 ```jsx
 class StarRating extends React.Component {
@@ -20,22 +20,19 @@ class StarRating extends React.Component {
     this.hoverOver = this.hoverOver.bind(this);
     this.setRating = this.setRating.bind(this);
   }
-
   hoverOver(val) {
     this.setState(state => ({ selection: val }));
   }
-
-  setRating(val) {
+  setRating(event) {
+    const val = event.target.getAttribute('star-id') || this.state.rating;
     this.setState(state => ({ rating: val }));
   }
-
   render() {
-    function Star({ marked, starId, onHover, onRate }) {
+    function Star({ marked, starId, onHover }) {
       return (
         <span
           star-id={starId}
           style={{ color: '#ff9933' }}
-          onClick={() => onRate(starId)}
           onMouseEnter={() => onHover(starId)}
         >
           {marked ? '\u2605' : '\u2606'}
@@ -43,7 +40,7 @@ class StarRating extends React.Component {
       );
     }
     return (
-      <div onMouseOut={() => this.hoverOver(0)}>
+      <div onMouseOut={() => this.hoverOver(0)} onClick={this.setRating}>
         {Array.from({ length: 5 }, (v, i) => i + 1).map(v => (
           <Star
             starId={v}
@@ -54,7 +51,6 @@ class StarRating extends React.Component {
                 : this.state.rating >= v
             }
             onHover={this.hoverOver}
-            onRate={this.setRating}
           />
         ))}
       </div>