[jQuery] adding dynamic buttons to code slider

[macsig]

Hello guys,

I'm trying to make some changes to the "coda slider" created by Remy
at jqueryfordesigner.com but I have hard time so figure out that is
wrong with my code. I hope someone can help me out with this issue.

I want to change dynamically the left and right buttons according to
the current panel. (something like they guys at macheist.com have done
for their 3rd bundle website)

Here my HTML:

<div id="slider">

        <div id="left_scroller"></div>

  <ul class="navigation">
    <li><a href="#first" id="1">Server Side</a></li>
    <li><a href="#second" id="2">Client Side</a></li>
    <li><a href="#third" id="3">Mobile</a></li>
  </ul>

  <div class="scroll">

    <div class="scrollContainer">

      <div class="panel" id="first">lorem ...</div>
      <div class="panel" id="second">lorem ...</div>
      <div class="panel" id="third">lorem ...</div>
    </div>
  </div>

   <div id="right_scroller"></div>
</div>


Here my JS:

// when the DOM is ready...
$(document).ready(function () {

    var $panels = $('#slider .scrollContainer > div');
    var $container = $('#slider .scrollContainer');

    // if false, we'll float all the panels left and fix the width
    // of the container
    var horizontal = true;

    // float the panels left if we're going horizontal
    if (horizontal) {
        $panels.css({
            'float' : 'left',
            'position' : 'relative' // IE fix to ensure overflow is
hidden
        });

        // calculate a new width for the container (so it holds all
panels)
        $container.css('width', $panels[0].offsetWidth *
$panels.length);
    }

    // collect the scroll object, at the same time apply the hidden
overflow
    // to remove the default scrollbars that will appear
    var $scroll = $('#slider .scroll').css('overflow', 'hidden');


    // handle nav selection
    function selectNav() {
        $(this)
            .parents('ul:first')
                .find('a')
                    .removeClass('selected')
                .end()
            .end()
            .addClass('selected');
        }

        // handle buttons
        function updateButtons() {
                //$(this)
                        //apply our left + right buttons
                        // images have to be 0 to n-1 where n is number of 
panels

                        var left_index = (parseInt(this.id) - 1) % $('#slider 
.navigation
li').length;
                        var right_index = (parseInt(this.id) + 1) % $('#slider 
.navigation
li').length;

                        $('#left_scroller').html('<img class="scrollButtons 
left"
src="images/'+left_index+'.png" />');
                        $('#right_scroller').html('<img class="scrollButtons 
right"
src="images/'+right_index+'.png" />');
    }

    $('#slider .navigation').find('a').click(selectNav);
    $('#slider .navigation').find('a').click(updateButtons);

    // go find the navigation link that has this target and select the
nav
    function trigger(data) {
        var el = $('#slider .navigation').find('a[href$="' + data.id +
'"]').get(0);

       selectNav.call(el);
       updateButtons.call(el);
    }

    if (window.location.hash) {
        trigger({ id : window.location.hash.substr(1) });
    } else {
        $('ul.navigation a:first').click();
    }

    // offset is used to move to *exactly* the right place, since I'm
using
    // padding on my example, I need to subtract the amount of padding
to
    // the offset.  Try removing this to get a good idea of the effect
    var offset = parseInt((horizontal ?
        $container.css('paddingTop') :
        $container.css('paddingLeft'))
        || 0) * -1;


    var scrollOptions = {
        target: $scroll, // the element that has the overflow

        // can be a selector which will be relative to the target
        items: $panels,

        navigation: '.navigation a',

        // selectors are NOT relative to document, i.e. make sure
they're unique
        prev: 'img.left',
        next: 'img.right',

        // allow the scroll effect to run both directions
        axis: 'xy',

        onAfter: trigger, // our final callback

        offset: offset,

        // duration of the sliding effect
        duration: 500,

        // easing - can be used with the easing plugin:
        // http://gsgd.co.uk/sandbox/jquery/easing/
        easing: 'swing'
    };

    // apply serialScroll to the slider - we chose this plugin because
it
    // supports// the indexed next and previous scroll along with
hooking
    // in to our navigation.
    $('#slider').serialScroll(scrollOptions);

    // now apply localScroll to hook any other arbitrary links to
trigger
    // the effect
    $.localScroll(scrollOptions);

    // finally, if the URL has a hash, move the slider in to
position,
    // setting the duration to 1 because I don't want it to scroll in
the
    // very first page load.  We don't always need this, but it
ensures
    // the positioning is absolutely spot on when the pages loads.
    scrollOptions.duration = 1;
    $.localScroll.hash(scrollOptions);

});

I have removed the static button creation and I have added the
function updateButtons in order to change the buttons.

Here the issue I’m facing right now: the buttons work only when the
page is loaded; after clicking on one of them or on a navigation link
the buttons don’t work anymore. The images are correctly updated if I
use the navigation links but nothing happens when I click them.


Adding a console log message within updateButtons I have noticed the
function is executed twice when i click a navigation link but only
once when I click the buttons. Perhaps this information can give you
some clues about the problem.

Any idea about why this happens?


Thanks for any help. I do appreciate it.

Sig