From: Maggie Costello Wachs <fg.maggie@gmail.com>
Date: Thu, 11 Dec 2008 17:46:29 +0000 (+0000)
Subject: updated #3650:  noted that inline position style should be removed from range element... 
X-Git-Tag: 1.6rc3~315
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=4c83fcc777ff52534459aea214a63797e0141391;p=jquery-ui.git

updated #3650:  noted that inline position style should be removed from range element (ui-slider-range)

ui.slider.css - added position style, fixed range display issues for IE6

ui.core.css - fixed clearfix class to work in IE7, Opera
---

diff --git a/tests/static/slider.html b/tests/static/slider.html
index b0cdb064c..ef3479405 100644
--- a/tests/static/slider.html
+++ b/tests/static/slider.html
@@ -15,7 +15,7 @@
 <div class="ui-slider ui-widget ui-widget-content ui-corner-all">
 	<a href="#" class="ui-slider-handle ui-state-default ui-corner-all" style="left: 15%;"></a>
 	<a href="#" class="ui-slider-handle ui-state-default ui-corner-all"  style="left: 30%;"></a>	
-	<div class="ui-slider-range ui-state-default" style="position: absolute; left: 15%; width: 15%;"></div>
+	<div class="ui-slider-range ui-state-default" style="left: 15%; width: 15%;"></div>
 </div>
 
 
@@ -25,7 +25,7 @@
 <div class="ui-slider ui-slider-vertical ui-widget ui-widget-content ui-corner-all" style="height: 100px;">
 	<a href="#" class="ui-slider-handle ui-state-default ui-corner-all" style="top: 15%;"></a>
 	<a href="#" class="ui-slider-handle ui-state-default ui-corner-all"  style="top: 60%;"></a>	
-	<div class="ui-slider-range ui-state-default" style="position: absolute; top: 15%; height: 45%;"></div>
+	<div class="ui-slider-range ui-state-default" style="top: 15%; height: 45%;"></div>
 </div>
 
 
diff --git a/themes/base/ui.core.css b/themes/base/ui.core.css
index 74721eb2e..778814574 100644
--- a/themes/base/ui.core.css
+++ b/themes/base/ui.core.css
@@ -9,7 +9,13 @@
 .ui-helper-hidden { display: none; }
 .ui-helper-hidden-accessible { position: absolute; left: -99999999px; }
 .ui-helper-reset { margin: 0; padding: 0; border: 0; outline: 0; line-height: 1.3; text-decoration: none; font-size: 100%; list-style: none; }
+
 .ui-helper-clearfix:after { content: "."; display: block; height: 0; clear: both; visibility: hidden; }
-.ui-helper-clearfix { display: inline-block; display: block; }
-* html .ui-helper-clearfix { height: 1%; }
+.ui-helper-clearfix { display: inline-block; }
+
+/* DO NOT REMOVE THIS COMMENT - required for clearfix to work in Opera \*/
+* html .ui-helper-clearfix { height:1%; }
+.ui-helper-clearfix { display:block; }
+/* end required block */
+
 .ui-helper-zfix { width: 100%; height: 100%; top: 0; left: 0; position: absolute; opacity: 0; filter:Alpha(Opacity=0); }
\ No newline at end of file
diff --git a/themes/base/ui.slider.css b/themes/base/ui.slider.css
index 8c642731e..4f434bd3e 100644
--- a/themes/base/ui.slider.css
+++ b/themes/base/ui.slider.css
@@ -1,8 +1,9 @@
 /* Slider
 ----------------------------------*/
 .ui-slider { height: .8em; position: relative; }
-.ui-slider-handle { position: absolute;z-index: 2;top: -.3em;width: 1.2em;height: 1.2em; }
-.ui-slider-range { height: .8em;position: absolute;border-left: 0;border-right: 0;top: -1px;z-index: 1;opacity:.7;filter:Alpha(Opacity=70); }
+.ui-slider-handle { position: absolute; z-index: 2; top: -.3em; width: 1.2em; height: 1.2em; }
+.ui-slider-range { position: relative; z-index: 1; top: 0; height: 100%; font-size:1%; display:block; border:0; opacity:.7; filter:Alpha(Opacity=70); }
+
 .ui-slider-vertical { height: auto; width: .8em; }
 .ui-slider-vertical .ui-slider-handle { top: auto; left: -.3em; }
-.ui-slider-vertical .ui-slider-range { height: auto; width: .8em; border-left: 1px; border-right: 1px; border-top: 0; border-bottom: 0; top: auto; left: 0; }
\ No newline at end of file
+.ui-slider-vertical .ui-slider-range { top: auto; left: 0; height: auto; width:100%; }
\ No newline at end of file