Question

I’m using jQuery Tools (http://jquerytools.org/) and cannot get the below function to accept a passed parameter. I’m not proficient in javascript or jquery and cannot find a solution anywhere that will make this work for the below code. Thank you for any help!

Current setup:

<a href='javascript:popup();'>Text Link That Calls Below Function</a>



<script>
function popup() {
if ($("#facebox").hasClass("init")) {
    $("#facebox").overlay().load();
}
else {
    $("#facebox").addClass("init");
    $("#facebox").overlay({

     // custom top position
     top: 260,

     mask: { color: '#838383',
         loadSpeed: 200,
         opacity: 0.5
     },  
     closeOnClick: true,
     load: true
    });
}
}
</script>

I would like it to do something like this…

<a href='javascript:popup(apples);'>Text Link That Calls Below Function</a>



<script>
function popup(choosebox) {
if ($("#choosebox").hasClass("init")) {
    $("#choosebox").overlay().load();
}
else {
    $("#choosebox").addClass("init");
    $("#choosebox").overlay({

     // custom top position
     top: 260,

     mask: { color: '#838383',
         loadSpeed: 200,
         opacity: 0.5
     },  
     closeOnClick: true,
     load: true
    });
}
}
</script>

Answer 1

You need to pass a string as an arguement, unless you have a variable named apple defined above (var apples; ). Try changing it like below,

<a href='javascript:popup("apples");'>Text Link That Calls Below Function</a>

Note the quotes surrounding the popup("apples")

Since you are using jQuery, you can do it nicely like below,

HTML:

<a href='javascript:void(0)' class="aLink" >Text Link That Calls Below Function</a>

JS:

$(function () {
    $('.aLink').click(function () {
        popup("apples");
    });
});

Also I think you may need to change your selector like below,

function popup(choosebox) {
    var $choosebox = $("#" + choosebox);

    if ($choosebox.hasClass("init")) {
        $choosebox.overlay().load();
    }
    else {
        $choosebox.addClass("init");
        $choosebox.overlay({
    //..rest of your code