Jquery - Dynamically pass parameters to function

Question

I'm not even sure I'm asking or going about this the right way but it's probably easier if I just show it. Basically, I'm trying to have run an attr of each div as a parameter through a function and have it return a different result based on that div's attr.

The example, as you can see, is a group of dropdowns that appear when you click on a link in the container div. If you make a selection it saves that as a attr in the parent div. The problem arises when you click out, then back in on the container ... instead of reshowing each dropdown with the appropriate default or selection showing, it just mirrors the result of the a next to it.

http://jsfiddle.net/nosfan1019/b7F6x/5/

TIA

Try to add some comments to your example, it's hard to read, imho — ted
– ted, Commented Aug 2, 2012 at 0:28
Try renaming your variables to something more legible than asdf and eeee. — Blender
– Blender, Commented Aug 2, 2012 at 0:32
Sorry about that. Went through and added both comments and better variables. jsfiddle.net/nosfan1019/b7F6x/5 — Ryan Grush
– Ryan Grush, Commented Aug 2, 2012 at 0:41

dsh · Accepted Answer · 2012-08-02 00:58:33Z

2

I inserted some console.log() statements to see what was happening with your various jQuery selectors. I observe the following:

when I click in the first "click" node, _container is "top one"
thus in your iteration of the three divs, you select both divs with class 'dd' contained in the div with class 'top one'
the parameters _attr and _parent that you pass to your function select() are the same for each node that is processed, giving the same result for both 'dd' boxes.

I think you want to change the selectors you use to locate the nodes to modify.

answered Aug 2, 2012 at 0:58

dsh

12.3k3 gold badges37 silver badges53 bronze badges

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

3 Comments

Ryan Grush Over a year ago

Thats what I was trying to describe in a sense. So its not a matter of passing some form of an array as a parameter, you don't think?

dsh Over a year ago

I don't think the problem is parameters and function calling, I think it is just selecting which objects you want to act on. If you comment out the line in the else: //$(this).find('.dd').empty(); does that give you the desired behavior? That else explicitly clears both divs in the other two "top" divs.

Ryan Grush Over a year ago

Not entirely, but that might have to work. The idea is that they hide or go to plain text when another part of the page is engaged. Oh well ... I appreciate the help

ted · Accepted Answer · 2012-08-02 02:48:51Z

foo = foo.find('.dropdown-toggle').html(_new + '<b class="caret"></b>');

with this line you get two divs and hence you've change both values(in case the value was chosen from the droplist).

To restore selected values correctly:

function modified(_select) {
    console.log("modify");
    foo = $('#box').html();
    foo = $(_select).html(foo);
    // iterate on collection to restore selected value from selection tag;
    foo.filter("div[selection]").each(function(i, v){
        var selected = $(v).attr('selection');
        $(v).find('.dropdown-toggle').html(selected + '<b class="caret"></b>');                
    });                
}

Then, it's needed to be checked if any of parentDiv has [selection] attr:

if($(y).filter("div[selection]").length > 0){
    return modified(y);
}

http://jsfiddle.net/b7F6x/50/

Collectives™ on Stack Overflow

Jquery - Dynamically pass parameters to function

2 Answers 2

3 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

3 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related