jQuery Slide Toggle function not called

Question

I am having trouble with jQuery Slide Toggle.

$(document).ready(function() {
    $("a").click(function() {
    $(this).parent().parent().children("p").Toggle(400);
    return false;
    });
    });

I have a bunch of posts on the page pulled from a database, and they are structured like this (pseudo HTML)

<div post>
    <div float-left location/timeInfo></div>
    <div float-right bandInfo><a>slideToggleLink</a></div>
    <p>theToggledElement</p>
</div>

When I click the link, the jQuery function is not called, but I cannot see what the problem is.

What do you think pseudo HTML is? I didn't want to post the whole thing so I used a pseudo mark-up to give the gist of the structure of the elements.... because I believed that it was the structure which was part of the problem, whereas now we know it isn't. — Luke
– Luke, Commented Feb 11, 2012 at 1:10

SLaks · Accepted Answer · 2012-02-11 00:42:27Z

1

jQuery does not have a Toggle() method.

You mean slideToggle().

answered Feb 11, 2012 at 0:42

SLaks

891k182 gold badges1.9k silver badges2k bronze badges

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

5 Comments

Sparky Over a year ago

jQuery does have a .toggle() method. It's just not spelled .Toggle().

Sparky Over a year ago

@elclanrs, yes, the OP wants it to slide. It does not change the fact that jQuery has a "toggle" method, contrary to the wording of this answer.

SLaks Over a year ago

@Sparky672: My answer is case-sensitive.

Sparky Over a year ago

Yes, your answer is technically correct. However some may read it and think there's no toggle method at all. I just made my comment as a point of clarification, more than correction.

Luke Over a year ago

.toggle() is actually a nicer animation. I'm keeping it. Thanks, guys!

elclanrs · Accepted Answer · 2012-02-11 00:47:35Z

1

$(this).parent().parent().children("p").Toggle(400);

Bad for performance and Toggle() does not exist.
Replace with:

$('#post').find('p').slideToggle(400);

answered Feb 11, 2012 at 0:47

elclanrs

94.2k21 gold badges137 silver badges171 bronze badges

5 Comments

elclanrs Over a year ago

Well, you should post your not pseudo html then.

Luke Over a year ago

Meh. It's JavaScript... I don't really care about performance.

elclanrs Over a year ago

Ok, well, bit by bit you'll end up with such bloated code that is not even gonna run.

Luke Over a year ago

How do you even know that? This could be the only JavaScript/jQuery in the whole site for all you know.

elclanrs Over a year ago

Do whatever you want. First, if you're asking this question is because your knowledge of jQuery/js is not as good as you think and second, performance should be a priority in any case. I'm just showing you what I believe is the right way to do it by experience. Answers like "Meh. It's JavaScript" are no good, and it'll bite you later on.

Sparky · Accepted Answer · 2012-02-11 00:52:12Z

1

You're asking about jQuery "Slide Toggle" but your code is using something called .Toggle(), which does not exist as it is spelled.

It should be .slideToggle() or just .toggle() (note: the lack of capital T in the second). However, there is no sliding/animation with .toggle().

edited Feb 11, 2012 at 0:52

answered Feb 11, 2012 at 0:45

Sparky

98.8k26 gold badges202 silver badges293 bronze badges

1 Comment

Luke Over a year ago

Damn. That was dumb. Thanks, anyway.

Collectives™ on Stack Overflow

jQuery Slide Toggle function not called

3 Answers 3

5 Comments

5 Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

5 Comments

5 Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related