bug fixes and updates for jQuery 1.6

jsteinberg · Jun 15, 2011 · 5310e35 · 5310e35
1 parent 28d393a
commit 5310e35
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 10 deletions.
diff --git a/.DS_Store b/.DS_Store
diff --git a/jquery.simpleCarousel.js b/jquery.simpleCarousel.js
@@ -35,10 +35,10 @@ jQuery simpleCarousel Plugin
       var $list = $(this),
           $items = $list.children();
           // Generated elements for the carousel
-      var $container = $('<div class="simpleCarouselContainer">'),
-          $wrapper = $('<div class="simpleCarouselWrapper">'),
-          $nextControl = $('<a href="#" class="simpleCarouselNext">next</a>'),
-          $prevControl = $('<a href="#" class="simpleCarouselPrev">prev</a>');
+      var $container = $('<div>').addClass('simpleCarouselContainer'),
+          $wrapper = $('<div>').addClass('simpleCarouselWrapper'),
+          $nextControl = $('<a>').attr('href', '#').addClass('simpleCarouselNext').text('next'),
+          $prevControl = $('<a>').addClass('simpleCarouselPrev').text('prev');
           // sizes
       var $currentPage = 1,
           $pages = Math.ceil($items.length/settings.showItems);
@@ -54,9 +54,9 @@ jQuery simpleCarousel Plugin
         .addClass(settings.customClass);
       $list.replaceWith($container);
 
-      $wrapper.appendTo($container);
       $nextControl.appendTo($container);
       $prevControl.appendTo($container);
+      $wrapper.appendTo($container);
 
       $wrapper
         .css({
@@ -114,10 +114,10 @@ jQuery simpleCarousel Plugin
         }
       };
 
-      function setControls(page) {
+      function setControls(page) {        
         if(page == $pages) {
           $nextControl.addClass('disabled');
-          $prevControl.removeClass('disabled');
+          $prevControl.addClass('disabled');
         } else if(page == 1) {
           $nextControl.removeClass('disabled');
           $prevControl.addClass('disabled');         
@@ -128,8 +128,8 @@ jQuery simpleCarousel Plugin
       };
 
       function updateControlHref(currentPage) { 
-        var nextPage,
-            prevPage;
+        var nextPage, prevPage;
+
         if(currentPage == $pages) {
           nextPage = 1;
           prevPage = $currentPage-1;

diff --git a/simpleCarousel_example.html b/simpleCarousel_example.html
@@ -106,7 +106,7 @@
       <li>ten</li>
     </ul>
 
-    <script type="text/javascript" src="../jquery/jquery-1.4.2.js"></script>
+    <script type="text/javascript" src="https://ajax.googleapis.com/ajax/libs/jquery/1.6.1/jquery.min.js"></script>
     <script type="text/javascript" src="jquery.simpleCarousel.js"></script>
     <script type="text/javascript">
       $(function() {