jQuery Rookie

Hi Cameron,

Let's go through this, you currently have the following JavaScript.

var $input = $('#label-text').val();
var $btn = $('#btn-labelr');
var $label = $('#label');

$btn.addEventListener('click', function() {
    $($label).text($input);
});

So what's going wrong?

1. Your $input variable is been assigned the default string in the input field which by default is an empty string, instead you want the object of the input field

2. jQuery objects have no prototype method called addEventListener as when you select an element in the DOM jQuery wraps it in an custom object rather than the standard DOM object which means you need to use one of jQuerys event methods to bind your click event.

3. As per my first point you've assigned an empty string to $input therefore the text method isn't receiving the new text from the input field, also wrapping $label within another jQuery object is redundant as your $label variable already has the method text assigned to it's prototype.

How do you fix it?

See the below, as you can see .val() has been removed from the $input variable, $btn now binds an event using the on method and text method now retrieves the value of $input every time the button is clicked.

var $input = $('#label-text');
var $btn = $('#btn-labelr');
var $label = $('#label');

$btn.on('click', function() {
    $label.text($input.val());
});

Hope that helps and happy coding.