Update <Site> component to use ES6 syntax.

[mtias]

No description provided.

[retrofox]

are you avoiding camelCase on purpose?

[mtias]

I don't think it's necessary here.

[retrofox]

BTW I got these eslint warning

it comes from eslint-plugin-react module.

My suggestion:

const setStartToottip = startTooltip => this.setState( { startTooltip } );

return (
  <button
    className="site__star"
    onClick={ this.starSite }
    onMouseEnter={ setStartTooltip( true ) }
    onMouseLeave={ setStartTooltip( false ) }
    // ...

[gwwar]

If we're bothering to work around the warning I'd suggest doing it properly. Basically we don't want to create functions in a render method. One way to fix this would be to add:

enableStarTooltip() {
    this.setState( { starTooltip: true } );
},
disableStarTooltip() {
    this.setState( { starTooltip: false } );
},
render() {
   return ( <button onMouseEnter={ this.enableStarTooltip } onMouseLeave={ this.disableStarTooltip } /> );
}

[gwwar]

👍 changes look good to 🚢 after a rebase

[blowery]

@gwwar Marking this back as Needs Review since I rebased it and added the bound methods to get rid of the JSX-bind linting warnings.

[gwwar]

; ?

[gwwar]

🚢 after adding in the missing ;

[blowery]

@gwwar nice catch! fixed in an amended commit.