[jQuery] Problem ze zrobieniem nawigacji w sliderze Opcje

[lamcpp]

Witam, skrypt slidera, który podaje poniżej. Skrypt działa dobrze poza przyciskami next i previous,
kod odpowiedzialny za te przyciski to:

[HTML] pobierz, plaintext 
           $("#button a#next").click(function(event){
            var pos = $(.menuItem).prevAll('.menuItem').length;
	    $('#slides').stop().animate({marginLeft:-positions[pos]+'px'},450).next();
            event.preventDefault();
          });
 
          $("#button a#previous").click(function(event){
          var pos = $(.menuItem).prevAll('.menuItem').length;
	  $('#slides').stop().animate({marginLeft:-positions[pos]+'px'},450).prev();
            event.preventDefault();
          });
[HTML] pobierz, plaintext 

Jednak kod ten nie działa w odniesieniu do całości kodu. Zasada działania tego kawałka kodu powinna być taka, że po kliknięciu przycisku next bądź previous przez użytkownika, do zmiennej "pos" przypisany zostanie numer pozycji, dzięki ktorej w następnej linice, będziemy ustawiac dany slajd o ileś px w lewo (margin-left) dzięki czemu slajdy będą się zmmieniały po sobie. Niestety przyciski next i previous nie działają


html:

[HTML] pobierz, plaintext 
<div id="main">
 
<div id="gallery">
 
<div id="slides">
 
<div class="slide"><img src="img/sample_slides/macbook.jpg" width="920" height="400" /></div>
<div class="slide"><img src="img/sample_slides/iphone.jpg" width="920" height="400" /></div>
<div class="slide"><img src="img/sample_slides/imac.jpg" width="920" height="400" /></div>
 
</div>
 
<div id="menu">
 
<ul>
<li class="fbar">&nbsp;</li><li class="menuItem"><a href=""><img src="img/sample_slides/thumb_macbook.png" /></a></li><li class="menuItem"><a href=""><img src="img/sample_slides/thumb_iphone.png" /></a></li><li class="menuItem"><a href=""><img src="img/sample_slides/thumb_imac.png" /></a></li>
</ul>
 
</div>
div id="button">
              <a href="#" id="previous">previous</a>
              <a href="#" id="next">next</a>
            </div>
 
</div>
 
</div>
[HTML] pobierz, plaintext 

Css:

[HTML] pobierz, plaintext 
body,h1,h2,h3,p,quote,small,form,input,ul,li,ol,label{
	/* Page reset */
	margin:0px;
	padding:0px;
}
 
body{
	/* Setting default text color, background and a font stack */
	color:#444444;
	font-size:13px;
	background: #f2f2f2;
	font-family:Arial, Helvetica, sans-serif;
}
 
/* Gallery styles */
 
#gallery{
	/* CSS3 Box Shadow */
	-moz-box-shadow:0 0 3px #AAAAAA;
	-webkit-box-shadow:0 0 3px #AAAAAA;
	box-shadow:0 0 3px #AAAAAA;
 
	/* CSS3 Rounded Corners */
 
	-moz-border-radius-bottomleft:4px;
	-webkit-border-bottom-left-radius:4px;
	border-bottom-left-radius:4px;
 
	-moz-border-radius-bottomright:4px;
	-webkit-border-bottom-right-radius:4px;
	border-bottom-right-radius:4px;
 
	border:1px solid white;
 
	background:url(img/panel.jpg) repeat-x bottom center #ffffff;
 
	/* The width of the gallery */
	width:920px;
	overflow:hidden;
}
 
#slides{
	/* This is the slide area */
	height:400px;
 
	/* jQuery changes the width later on to the sum of the widths of all the slides. */
	width:920px;
	overflow:hidden;
}
 
.slide{
	float:left;
}
 
#menu{
	/* This is the container for the thumbnails */
	height:45px;
}
 
ul{
	margin:0px;
	padding:0px;
}
 
li{
	/* Every thumbnail is a li element */
	width:60px;
	display:inline-block;
	list-style:none;
	height:45px;
	overflow:hidden;
}
 
li.inact:hover{
	/* The inactive state, highlighted on mouse over */
	background:url(img/pic_bg.png) repeat;
}
 
li.act,li.act:hover{
	/* The active state of the thumb */
	background:url(img/active_bg.png) no-repeat;
}
 
li.act a{
	cursor:default;
}
 
.fbar{
	/* The left-most vertical bar, next to the first thumbnail */
	width:2px;
	background:url(img/divider.png) no-repeat right;
}
 
li a{
	display:block;
	background:url(img/divider.png) no-repeat right;
	height:35px;
	padding-top:10px;
}
 
a img{
	border:none;
}
[HTML] pobierz, plaintext 

i najważniejsze czyli kod jquery:

[XML] pobierz, plaintext 
$(document).ready(function(){
	/* This code is executed after the DOM has been completely loaded */
 
	var totWidth=0;
	var positions = new Array();
 
	$('#slides .slide').each(function(i){
		/* Loop through all the slides and store their accumulative widths in totWidth */
		positions[i]= totWidth;
		totWidth += $(this).width();
 
		/* The positions array contains each slide's commulutative offset from the left part of the container */
 
		if(!$(this).width())
		{
			alert("Please, fill in width & height for all your images!");
			return false;
		}
	});
 
	$('#slides').width(totWidth);
 
	/* Change the cotnainer div's width to the exact width of all the slides combined */
 
	$('#menu ul li a').click(function(e){
 
		/* On a thumbnail click */
		$('li.menuItem').removeClass('act').addClass('inact');
		$(this).parent().addClass('act');
 
		var pos = $(this).parent().prevAll('.menuItem').length;
 
		$('#slides').stop().animate({marginLeft:-positions[pos]+'px'},450);
		/* Start the sliding animation */
 
		e.preventDefault();
		/* Prevent the default action of the link */
	});
 
           $("#button a#next").click(function(event){
            var pos = $(.menuItem).prevAll('.menuItem').length;
	    $('#slides').stop().animate({marginLeft:-positions[pos]+'px'},450).next();
            event.preventDefault();
          });
 
          $("#button a#previous").click(function(event){
          var pos = $(.menuItem).prevAll('.menuItem').length;
	  $('#slides').stop().animate({marginLeft:-positions[pos]+'px'},450).prev();
            event.preventDefault();
          });
 
	$('#menu ul li.menuItem:first').addClass('act').siblings().addClass('inact');
	/* On page load, mark the first thumbnail as active */
});
[XML] pobierz, plaintext 

[erix]

Wstaw to gdzieś działające albo nie licz na odzew.

[Rid]

Nie Masz otworzonego diva brakuje " <" powinno być:

[HTML] pobierz, plaintext 
</div>
<div id="button">
<a href="#" id="previous">previous</a>
<a href="#" id="next">next</a>
</div>
[HTML] pobierz, plaintext 

poza tym akcje ustawiasz na przciski <a> to po co w jquery $("#button a#next") nie wystarczy $("#next")?


Ten post edytował Rid 7.09.2011, 15:03:39

[bulias]

[HTML] pobierz, plaintext 
line 20 
div id="button">
[HTML] pobierz, plaintext 

To co rzuciło mi się po przejrzeniu źródła to temu znacznikowi brakuje,

[HTML] pobierz, plaintext 
<
[HTML] pobierz, plaintext 

czyli

[HTML] pobierz, plaintext 
<div id="button">
[HTML] pobierz, plaintext 

oraz

[PHP] pobierz, plaintext 
var pos = $('.menuItem').prevAll('.menuItem').length;
...
...
var pos = $('.menuItem').prevAll('.menuItem').length;
[PHP] pobierz, plaintext 

Ten post edytował bulias 7.09.2011, 15:08:06