? slider-css-opti.patch
? images/scroll.png
Index: slider.module
===================================================================
RCS file: /cvs/drupal-contrib/contributions/modules/slider/slider.module,v
retrieving revision 1.1.2.5
diff -u -p -r1.1.2.5 slider.module
--- slider.module	6 Mar 2009 09:47:54 -0000	1.1.2.5
+++ slider.module	25 Sep 2010 16:47:26 -0000
@@ -76,11 +76,6 @@ function slider_nodeapi(&$node, $op, $te
 function _slider_includes() {
   $modulepath = drupal_get_path('module', 'slider');
   drupal_add_css($modulepath . '/css/slider.css');
-
-  // pass through the dynamic path so it can be accessed from the jquery
-  $js = "var path_to_slider = '". base_path() . $modulepath . "';";
-  drupal_add_js($js, 'inline');
-
   drupal_add_js($modulepath . '/js/jquery.localscroll-min.js');
   drupal_add_js($modulepath . '/js/jquery.scrollTo-min.js');
   drupal_add_js($modulepath . '/js/jquery.serialScroll-min.js');
Index: css/slider.css
===================================================================
RCS file: /cvs/drupal-contrib/contributions/modules/slider/css/slider.css,v
retrieving revision 1.1
diff -u -p -r1.1 slider.css
--- css/slider.css	15 Oct 2008 10:04:26 -0000	1.1
+++ css/slider.css	25 Sep 2010 16:47:26 -0000
@@ -1,84 +1,93 @@
-/* $Id: slider.css,v 1.1 2008/10/15 10:04:26 marktheunissen Exp $ */
-
-/**
- * Standard CSS for the slider.
- * Overrride these values in your theme's CSS file
- */
-
-#sliderwrapper {
-  width: 760px;
-  margin: 0 auto;
-}
-
-#slider {
-  width: 620px;
-  margin: 0 auto;
-  position: relative;
-}
-
-.scroll {
-  height: 250px;
-  width: 620px;
-  overflow: auto;
-  overflow-x: hidden;
-  position: relative;
-  clear: left;
-}
-
-.scrollContainer div.panel {
-  padding: 20px;
-  height: 210px;
-  width: 580px;
-}
-
-ul.slidenav {
-  list-style: none;
-  margin: 0;
-  padding: 0;
-  padding-bottom: 9px;
-}
-
-ul.slidenav .slidetab-active {
-  background-color: #e6e6e6;
-}
-
-ul.slidenav li {
-  display: inline;
-  margin-right: 10px;
-}
-
-ul.slidenav a {
-  padding: 10px;
-  color: #000;
-  text-decoration: none;
-}
-
-ul.slidenav a:hover {
-  background-color: #e6e6e6;
-}
-
-ul.slidenav a.selected {
-  background-color: #fff;
-}
-
-ul.slidenav a:focus {
-  outline: none;
-}
-
-.scrollButtons {
-  position: absolute;
-  top: 150px;
-  cursor: pointer;
-}
-
-.scrollButtons.left {
-  left: -20px;
-}
-
-.scrollButtons.right {
-  right: -20px;
-}
-
-.hide {
-  display: none;
-}
+/* $Id: slider.css,v 1.1 2008/10/15 10:04:26 marktheunissen Exp $ */
+
+/**
+ * Standard CSS for the slider.
+ * Overrride these values in your theme's CSS file
+ */
+
+#sliderwrapper {
+  width: 760px;
+  margin: 0 auto;
+}
+
+#slider {
+  width: 620px;
+  margin: 0 auto;
+  position: relative;
+}
+
+.scroll {
+  height: 250px;
+  width: 620px;
+  overflow: auto;
+  overflow-x: hidden;
+  position: relative;
+  clear: left;
+}
+
+.scrollContainer div.panel {
+  padding: 20px;
+  height: 210px;
+  width: 580px;
+}
+
+ul.slidenav {
+  list-style: none;
+  margin: 0;
+  padding: 0;
+  padding-bottom: 9px;
+}
+
+ul.slidenav .slidetab-active {
+  background-color: #e6e6e6;
+}
+
+ul.slidenav li {
+  display: inline;
+  margin-right: 10px;
+}
+
+ul.slidenav a {
+  padding: 10px;
+  color: #000;
+  text-decoration: none;
+}
+
+ul.slidenav a:hover {
+  background-color: #e6e6e6;
+}
+
+ul.slidenav a.selected {
+  background-color: #fff;
+}
+
+ul.slidenav a:focus {
+  outline: none;
+}
+
+.scrollButtons {
+  position: absolute;
+  top: 150px;
+  cursor: pointer;
+  width: 16px;
+  height: 16px;
+  background: transparent no-repeat url(../images/scroll.png) 0 -16px;
+  overflow: hidden;
+}
+
+.scrollButtons:hover {
+	opacity: .8;
+}
+
+.scrollButtons.slide-left {
+  left: -20px;
+}
+
+.scrollButtons.slide-right {
+  right: -20px;
+  background-position: 0 0;
+}
+
+.hide {
+  display: none;
+}
Index: js/slider.js
===================================================================
RCS file: /cvs/drupal-contrib/contributions/modules/slider/js/slider.js,v
retrieving revision 1.1.2.2
diff -u -p -r1.1.2.2 slider.js
--- js/slider.js	25 Feb 2009 11:52:11 -0000	1.1.2.2
+++ js/slider.js	25 Sep 2010 16:47:26 -0000
@@ -25,8 +25,8 @@ Drupal.behaviors.initSlider = function(c
 
   // apply our left + right buttons
   scroll
-    .before('<img class="scrollButtons left" src="' + path_to_slider + '/images/scroll_left.png" />')
-    .after('<img class="scrollButtons right" src="' + path_to_slider + '/images/scroll_right.png" />');
+    .before('<div class="scrollButtons slide-left" />')
+    .after('<div class="scrollButtons slide-right" />');
 
   // handle nav selection
   function selectNav() {
@@ -70,8 +70,8 @@ Drupal.behaviors.initSlider = function(c
     navigation: '.slidenav a',
 
     // selectors are NOT relative to document, i.e. make sure they're unique
-    prev: 'img.left',
-    next: 'img.right',
+    prev: '.slide-left',
+    next: '.slide-right',
 
     // allow the scroll effect to run both directions
     axis: 'xy',