Ex 7 Disable the button 1


#1

The code works fine except for the else portion, the post button seems to maintain the disabled class with the following code
var main = function() {
$('.btn').addClass('disabled');
$('.status-box').keyup(function() {
var postlength = $(this).val().length;
var charactersLeft = 140 - postlength;
$('.counter').text(charactersLeft);

  if (charactersLeft < 0) {
   $('.btn').addClass('disabled');
  }

  else if (charactersLeft = 140) {
   $('.btn').addClass('disabled');
  }
  else {
   $('.btn').removeClass('disabled');
  }

});

$('.btn').click(function() {

var post = $('.status-box').val();
$('<li>').text(post).prependTo('.posts');
$('.status-box').val('');
$('.counter').text('140');

});

}

$(document).ready(main)

I found a way for the code to work but I do not understand why, it works when I add {} after the else which comes out as following:

var main = function() {
$('.btn').addClass('disabled');
$('.status-box').keyup(function() {
var postlength = $(this).val().length;
var charactersLeft = 140 - postlength;
$('.counter').text(charactersLeft);

  if (charactersLeft < 0) {
   $('.btn').addClass('disabled');
  }

  else if (charactersLeft = 140) {
   $('.btn').addClass('disabled');
  }
  else {}
   $('.btn').removeClass('disabled');

});

$('.btn').click(function() {

var post = $('.status-box').val();
$('<li>').text(post).prependTo('.posts');
$('.status-box').val('');
$('.counter').text('140');

});

}

$(document).ready(main)

I also tried removing the brackets entirely and it did not work either.

Thanks for the help,


#2

@mar_lewi,

The problem was in your

else if (charactersLeft = 140) {

_which should be a =comparison= like

else if (charactersLeft === 140) {

#3

@mar_lewi,

- - - - - - - - - - - - S t a t u s U p d at e - - - - - - - - - -

= = = = = = = D ocument O bject M odel = = = = =

                     2html
                       |
        |--------------+-----------|
      3head                      8body
        |                          |
  |-----|+----|                    |
  |     |     |                    |
4link 5link 6link                9div
                                   |
  |----------------|---------------+---|--------------|
  |                |                   |              |
10form           15div(.button-group 20ul         24script(2x)
   |                |  .pull-right)    (.posts)    25script
11div(.form-group)  |
   |                |
12textarea          |
    (.form-control  |
    .status-box)    |
                    |
         |----------+----|
       16p(.counter)   17a(.btn
         |               | .btn-primary)
        text            text
        node            node

+++++++++++++++++++++++++++++++++++++++++++
The charactersLeft variable
is created by substracting
the == length of the actual characters in the status-box ==
var postLength = $('.status-box').val().length ;
from
the == maximum characters allowed which is 140 ==
thus
var charactersLeft = 140 - postLength

Now you will have to check on 3 conditions:
1- charactersLeft < 0
You have put in more then 140 characters into the $('.status-box')
thus you will want to add a class-attribute with class-name disabled
to the HTML-Element which is allready carrying the class-name btn
using the jQuery-method addClass()

2- charactersLeft === 140
You have not put any characters into the $('.status-box')
thus you will want to add a class-attribute with class-name disabled
to the HTML-Element which is allready carrying the class-name btn
using the jQuery-method addClass()

3- in all other charactersLeft cases
you will want to make sure the btn
does NOT have the disabled class-name set.
Remove a class-attribute with class-name disabled
from the HTML-Element which is allready carrying the class-name btn
using the jQuery-method removeClass()

This you can achieve by setting up an IF ELSE-IF ELSE construct
like

if ( condition1 ) {
   //Begin of IF-codeblock
}
else if ( condition2 ) {
   // Begin of ELSE-IF-codeblock
}
else {
  //You do not use a condition !!
  //Begin of ELSE-codeblock
}

You have to integrate the above mentioned
into the FUNCTION-BODY
of the $('.status-box').keydown() Event-handler.

$('.status-box').keydown( function() {
   //Begin of FUNCTION-BODY

  // your code

  //End of FUNCTION-BODY
  }
  //END-of-keydown Eventhandler
);

questions---
why is class-name -disabled- set in the =main= function
http://www.codecademy.com/forum_questions/555071a551b8870e9a00056e

the build of status-update function...
http://www.codecademy.com/forum_questions/55503ca8d3292f8c2b000281