views:

972

answers:

1

I am building a page that is employing several different javascript elements and I seem to have run into a problem I haven't before (not surprising as I am new to javascript).

I have implemented the tutorial for the JQuery Coda Slider located here: http://jqueryfordesigners.com/coda-slider-effect/.

It seems that the sliding effect works when I run it it Firefox but not Chrome or Safari. Wondering if this is a common issue and if I am missing something obvious.

To help, I am attaching the code of the page I am working on as it may help you understand the scripts I am using.

<head>
    <title></title>

    <link rel="stylesheet" href="stylesheets/style.css" media="screen" />

    <script src="js/jquery-1.3.2.min.js" type="text/javascript"></script>
    <script type="text/javascript" src="http://cloud.github.com/downloads/malsup/cycle/jquery.cycle.all.2.73.js"&gt;&lt;/script&gt;
    <script src= "js/banner.js"type="text/javascript"></script>
    <script src="js/menu.js" type="text/javascript"></script>
    <script src="js/jquery.localscroll-1.2.7-min.js" type="text/javascript"></script>
    <script src="js/jquery.scrollTo-1.4.2-min.js" type="text/javascript"></script>
    <script src="js/jquery.serialScroll-1.2.2-min.js" type="text/javascript"></script>
    <script src="js/slider.js" type="text/javascript"></script>

</head>

EDIT: slider.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');

// apply our left + right buttons
$scroll
  .before('<img class="scrollButtons left" src="images/scroll_left.png" />')
  .after('<img class="scrollButtons right" src="images/scroll_right.png" />');

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

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

// 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);
}

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);

});
A: 

Why didn't you just copy "http://jqueryfordesigners.com/demo/coda-slider.html" Its a reduction of what you need. Remember you need these:

<script src="jquery-1.2.6.js" type="text/javascript"></script> 
<script src="jquery.scrollTo-1.3.3.js" type="text/javascript"></script> 
<script src="jquery.localscroll-1.2.5.js" type="text/javascript" charset="utf-8"></script> 
<script src="jquery.serialScroll-1.2.1.js" type="text/javascript" charset="utf-8"></script> 
<script src="coda-slider.js" type="text/javascript" charset="utf-8"></script> 

And the HTML should be similar (view source to find out). It works great in Chrome.

Mohamed Mansour
Isn't that what I have. I just copied the most recent dowloads of JQuery and the Scroll series file name to the head link. What am I missing?
bgadoci
Try copying and pasting http://jqueryfordesigners.com/demo/coda-slider.html it should work on all browsers. Without knowing your HTML or your reduced sample, it is unknown what the problem you are facing. Must be something small syntax error. Try validating it.
Mohamed Mansour