How to use a dynamic property value in a jQuery widget

Question

I have a javascript that returns a value whenever I choose an option from a dropdown menu.

   $(document).ready(function() {
        function calculateVals() { 
            //dropdown menus      
             var valuestart = $("select[name='timestart']").val();
             var valuestop = $("select[name='timestop']").val();

             //create date format          
             var timeStart = new Date("01/01/2007 " + valuestart).getHours();
             var timeEnd = new Date("01/01/2007 " + valuestop).getHours();

             var hourDiff = timeEnd - timeStart;                         

             return hourDiff;  

        }
        $("select").change(calculateVals);
        calculateVals();
    });

The script above returns a number which I need to add to a jquery plugin below. I need to add it to the max option but I get the error calculateVal is not defined.

$(function(){
    $("#slider").slider({
        range: "min",
        value:  8,
        min: 1,
        max: calculateVals(), //my attempt
        step: 1,
        slide: function(event, ui) {
            $("#amount").text(ui.value);
        },
         stop: function(event, ui) {
            $("#notifications").val(ui.value);
        }
    });
    $("#notifications").val( $("#slider").slider("value") );
});

@gdoron thanks i dont see it tho haha

CyberJunkie
– CyberJunkie

2012-06-14 22:35:20 +00:00
Commented Jun 14, 2012 at 22:35 — CyberJunkie
– CyberJunkie, Commented Jun 14, 2012 at 22:35

gdoron · Accepted Answer · 2012-06-14 23:17:32Z

2

Move the function to the global scope:

function calculateVals() {       
    //code here...
    return someResult;                                           
}

$(document).ready(function() {
        $("select").change(calculateVals);
    calculateVals();
});

And even better, learn what does DOM ready means, as you misuse it.

You should have only one ready handler.
Only code relevant to DOM elements should be there.

Update:

You can later on change the slider max with:

$( "#slider").slider("option", "max", calculateVals());

edited Jun 14, 2012 at 23:17

answered Jun 14, 2012 at 22:24

gdoron

151k59 gold badges302 silver badges376 bronze badges

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

8 Comments

CyberJunkie Over a year ago

Thank you! Would some extra code make this dynamic? When i click my dropdown menu and choose an option nothing happens.

gdoron Over a year ago

@CyberJunkie. I got the feeling your problem lies here //code here... :) and anyway I don't know what you call dynamic, nor what exactly is the current problem.

CyberJunkie Over a year ago

you are right! I added the missing code to the answer. It calculates an hour difference between 2 times i select from the 2 dropdown menus and returns the result. The result I'm trying to add to the jquery function in max option which alters a slider. The function executes only when i load the page

gdoron Over a year ago

@CyberJunkie. You can later on change the slider max with $( "#slider").slider("option", "max", calculateVals());

CyberJunkie Over a year ago

do you mean like this? $("select").change(function() { $( "#slider" ).slider("option", "max", calculateTime()); }); I tried with no success..

|

Gabe · Accepted Answer · 2012-06-14 22:23:23Z

2

Try defining the function out side the ready block:

function calculateVals() { 

             //code here...

             return someResult;                                          
}



$(document).ready(function() {
    $("select").change(calculateVals);
    calculateVals();
});

answered Jun 14, 2012 at 22:23

Gabe

50.6k29 gold badges147 silver badges182 bronze badges

1 Comment

CyberJunkie Over a year ago

Thanks I tried. I don't get an error but nothing happens when I click the select menu. someResult should return a number which should make something happen on change.

Collectives™ on Stack Overflow

How to use a dynamic property value in a jQuery widget

2 Answers 2

8 Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

8 Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related