views:

405

answers:

1

I'm trying to load random items into a div every few seconds, with a nice fadeOut/fadeIn transition between each load. Here's the code:

<html>
  <body>
    <div id="item"></div>
    <script type="text/javascript" src="http://ajax.googleapis.com/ajax/libs/jquery/1.3.2/jquery.min.js"&gt;&lt;/script&gt;
    <script type="text/javascript">
      // Load a random item
      var item = $('#item');
      function load_item() {
        item.fadeOut(5000, function() {
          item.load('http://dynamic.xkcd.com/comic/random/ #middleContent img', null, function() {
            item.fadeIn(5000);
          });
        });
      };

      // Load initial featured item
      load_item();

      // Schedule repeated loading
      setInterval(load_item, 15000);
    </script>
  </body>
</html>

This works fine the first time through, but on subsequent calls to load_item, the fadeOut() seems to stop working. It doesn't actually fade the #item div out, but jumps immediately into the callback function, ignoring the 5000 delay.

What am I doing wrong?

A: 
setInterval("load_item()",15000)

I know it looks funny to put it in quotes, but it works.

The other option is to say:

load_item = function(){...etc...};
setInterval(loadItem,1500)
Kit MacAllister
That is definitely not the proper syntax. You should avoid eval. Passing a function reference like Clay does is the preferred way.
Magnar