jQuery - replace loop by className

Question

function loadTextboxes()
{
    var textareas = document.getElementsByTagName('textarea');

    for(var i=0; i < textareas.length; i++)
    {
        if (textareas.item(i).className == "richtextbox")
        {
            richtextbox(textareas.item(i));
        }
    }
}

//window.attachEvent('onload',loadTextboxes);

$(document).ready(function() {
    //loadTextboxes(); // works ...
    $('.richtextbox').each(richtextbox(this));
});

A JavaScript function searches for textarea with class "richtextbox" and calls another function (not posted here) ... tried to do this with jQuery - does not work :-(

cletus · Accepted Answer · 2010-03-04 08:57:28Z

3

The problem is this line:

$('.richtextbox').each(richtextbox(this));

means you call richtextbox(this) and pass it's return value into each(). That won't work unless the function returns a function.

What I suspect you mean is:

$(function() {
  $("textarea.richtextbox").each(function() {
    richtextbox(this);
  });
});

That's the correct way to pass a named function as a parameter.

Ideally, this would be assumed by the function rather than passed in as an argument, which would allow you to shorten the code to:

$(function() {
  $("textarea.richtextbox").each(richtextbox);
});

edited Mar 4, 2010 at 8:57

answered Mar 4, 2010 at 8:27

cletus

627k169 gold badges922 silver badges945 bronze badges

Sign up to request clarification or add additional context in comments.

Collectives™ on Stack Overflow

jQuery - replace loop by className

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related