Question

I’m not very good with js, but basically I have a button

<%=button_to_function "✓", checkButton(), :class => "buttonGrey"%>

that is calling a js function I made. However I am getting errors in my syntax for the function.

function checkButton()
{
if (this.className="buttonGrey") {
    this.removeClass('buttonGrey');
    this.addClass('buttonGreen');
}

if (this.className="buttonGreen") {
   this.removeClass('buttonGreen');
   this.addClass('buttonGrey');
}
}

Answer 1

Use == to compare strings and not =.
Your button will always be buttonGrey because if it changes on the first if statement, it will be changed again on the second because it’s not inside a else statement.
Also you can use toggleClass to toggle classes.

function checkButton() {
    $(this).toggleClass("buttonGrey buttonGreen");
}