Question

I am a jQuery noobie, and have been trying to add a script to change the id of a div onClick.

Here is a jsfiddle example.

$(function accept() {
    $("div:scrollwrap").attr('scrollwrap','highlight');
});​

Thanks 🙂

Answer 1

The first parameter of attr should be the attribute name, not the current value:

$(function accept() {
    $("div#scrollwrap").attr('id','highlight');
});​

However, reading your jsFiddle code, you appear to have a class of highlight and not an ID. Here is my edited version with what I think you are trying to achieve.

Note that I have changed the following:

• Made the .hightlight class more specific by adding the ID, otherwise the highlight style will not override the original.
• Removed the inline onClick as you can do this within your script, which is considered best practice (see JS for the .click() addition)
• Changed the JS function to toggle the class, as I assume it should be invalidated if the user deselects the checkbox.

More resources (jQuery docs):

• .toggleClass()
• .click()
• .attr()