jQuery each this

Question

var slides = $(\".promo-slide\");
slides.each(function(key, value){
    if (key == 1) {
        this.addClass(\"first\");
    }
});

Why do I get an

Answer 1

You probably want to do:

$(this).addClass

Answer 2

Inside jQuery callback functions, this (and also value, in your example) refers to a DOM object, not a jQuery object.

var slides = $(".promo-slide");
slides.each(function(key, value){
    if (key == 0) { // NOTE: the key will start to count from 0, not 1!
        $(this).addClass("first"); // Or $(value).addClass("first");
//------^^----^       
    }
});

BUT: In your case, this is easier:

$(".promo-slide:first").addClass("first");

As an aside, I find it a useful convention to prefix variables that contain a jQuery object with a $:

var $slides = $(".promo-slide");
$slides.each( /* ... */ );