Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

React this.props is null

I have the following:

import React from 'react';
import {render} from 'react-dom';

class TShirt extends React.Component {
    render () {
        return <div className="thsirt">{this.props.name}</div>;
    }
}

class FirstName extends React.Component {
    propTypes: {
        name: React.PropTypes.string.isRequired
    }
    constructor(props) {
        super(props);
        this.state = {
            submitted: false
        };
    }
    getName () {
        var name = this.refs.firstName.value;
        this.setState(function() {
          this.props.action(name);
        });
    }
    handleSubmit (e) {
        e.preventDefault();
        this.setState({ submitted: true }, function() {
            this.props.actionNav('color');
        });
    }
    render () {
        if(!this.state.submitted){
            return (
                <div>
                    <h2>tell us your first name</h2>
                    <form>
                        <input 
                            type="text"
                            ref="firstName"
                            onChange={this.getName.bind(this)}
                        />
                        <div className="buttons-wrapper">
                            <a href="#">back</a>
                            <button onClick={this.handleSubmit.bind(this)}>continue</button>
                        </div>
                    </form>
                </div>
            );
        }
        else {
             return <PickColor color={this.props.colorVal} />;
        }
    }
}

class Link extends React.Component {
    setActiveClass () {
        if(this.props.el == this.props.activeClass){
            return 'active';
        }
    }
    render () {
        var active = this.setActiveClass();
        return (
            <li className={active}>{this.props.el}</li>
        );
    }
}

class Nav extends React.Component {
    render () {
        var links = ['name', 'color', 'design', 'share'],
            newLinks = [],
            that = this;
        links.forEach(function(el){
            newLinks.push(<Link activeClass={that.props.active} key={el} el={el} />);
        });
        return (
            <ul>
                {newLinks}
            </ul>
        );
    }
}

class PickColor extends React.Component {
    getColorValue(event) {
        console.log(event.target.getAttribute("data-color"));
        console.log(this);
        //this.props.color(event.target.getAttribute("data-color"));
    }
    render () {
        var colors = ['red', 'purple', 'yellow', 'green', 'blue'],
            colorsLink = [],
            that = this;

        colors.forEach(function(el){
            colorsLink.push(<li data-color={el} key={el} onClick={that.getColorValue} ref={el}>{el}</li>);
        });

        return (
            <ul>
                {colorsLink}
            </ul>
        );
    }

}

class App extends React.Component {
    constructor(props) {
        super(props);
        this.state = {
            name: '',
            color: '',
            active: ''
        };
        this.getName = this.getName.bind(this);
        this.setActiveNav = this.setActiveNav.bind(this);
        this.setColor = this.setColor.bind(this);
    }
    getName (tshirt) {
        this.setState({ name:tshirt })
    }
    setActiveNav (active) {
        this.setState({ active:active })
    }
    setColor (color) {
        this.setState({ color:color })
    }
    render () {
        return (
            <section className={this.state.color}>
                <Nav active={this.state.active} />
                <TShirt name={this.state.name} />
                <FirstName name={this.state.name} action={this.getName} actionNav={this.setActiveNav} colorVal={this.setColor} />
            </section>
        );
    }
}

render(<App/>, document.getElementById('app'));

Inside the "PickColor" component I am trying to do this:

getColorValue(event) {
    console.log(event.target.getAttribute("data-color"));
    console.log(this);
    //this.props.colorVal(event.target.getAttribute("data-color"));
}

however this is returning null on click so I can't go ahead ans use:

this.props.colorVal

like image 634
Aessandro Avatar asked Oct 18 '22 19:10

Aessandro


1 Answers

The solution was here:

class PickColor extends React.Component {
    getColorValue(event) {
        console.log(event.target.getAttribute("data-color"));
        this.props.color(event.target.getAttribute("data-color"));
    }
    render () {
        var colors = ['red', 'purple', 'yellow', 'green', 'blue'],
            colorsLink = [],
            that = this;

        colors.forEach(function(el){
            colorsLink.push(<li data-color={el} key={el} onClick={that.getColorValue.bind(that)} ref={el}>{el}</li>);
        });

        return (
            <ul>
                {colorsLink}
            </ul>
        );
    }
}

I had to bind "that" to the onClick inside the forEach loop

like image 146
Aessandro Avatar answered Oct 20 '22 09:10

Aessandro