Spinner: Use inline-element for better inline-block support. Removed float:left from...

diff --git a/themes/base/jquery.ui.spinner.css b/themes/base/jquery.ui.spinner.css
index 0a0a5ef6de781c86a4164cfbf93e3a34e333a8ff..711f4c885582d6847b2a9b7678feb1e0d01c0f16 100644 (file)
--- a/themes/base/jquery.ui.spinner.css
+++ b/themes/base/jquery.ui.spinner.css
@@ -1,7 +1,7 @@
 /* Spinner\r
 ----------------------------------*/\r
 .ui-spinner { position:relative; display: inline-block; overflow: hidden; padding: 0; vertical-align: middle; height: 1.8em; }\r
-.ui-spinner-input { border: none; background: none; padding: 0; margin: .2em 0; vertical-align: middle; float: left; margin-left: .4em; margin-right: 22px; }\r
+.ui-spinner-input { border: none; background: none; padding: 0; margin: .2em 0; vertical-align: middle; margin-left: .4em; margin-right: 22px; }\r
 .ui-spinner-button { width: 16px; height: 50%; font-size: .5em; padding: 0; margin: 0; z-index: 100; text-align: center; vertical-align: middle; position: absolute; cursor: default; display: block; overflow: hidden; right: 0; }\r
 .ui-spinner a.ui-spinner-button { border-top: none; border-bottom: none; border-right: none; } /* more specificity required here to overide default borders */\r
 .ui-spinner .ui-icon { position: absolute; margin-top: -8px; top: 50%; left: 0; } /* vertical centre icon */\r
@@ -9,4 +9,4 @@
 .ui-spinner-down { bottom: 0; }\r
 \r
 /* TR overrides */\r
-div.ui-spinner { background: none; }\r
+span.ui-spinner { background: none; }\r

diff --git a/ui/jquery.ui.spinner.js b/ui/jquery.ui.spinner.js
index 80ade7ba4bca2cbced01b76cde19b75d66277fe5..8be9bbd0c54502569f90adb81c4f366177c83ce5 100644 (file)
--- a/ui/jquery.ui.spinner.js
+++ b/ui/jquery.ui.spinner.js
@@ -70,12 +70,6 @@ $.widget('ui.spinner', {
                                        self.hovered = false;\r
                                });\r
 \r
-               // TODO: move to theme, ask FG how\r
-               // fix inline-block issues for IE. Since IE8 supports inline-block we need to exclude it.\r
-               if (!$.support.opacity && uiSpinner.css('display') == 'inline-block' && $.browser.version < 8) {\r
-                       uiSpinner.css('display', 'inline');\r
-               }\r
-\r
                this.element\r
                        .bind('keydown.spinner', function(event) {\r
                                if (self.options.disabled) {\r
@@ -206,7 +200,7 @@ $.widget('ui.spinner', {
        },\r
        \r
        _uiSpinnerHtml: function() {\r
-               return '<div role="spinbutton" class="ui-spinner ui-state-default ui-widget ui-widget-content ui-corner-all"></div>';\r
+               return '<span role="spinbutton" class="ui-spinner ui-state-default ui-widget ui-widget-content ui-corner-all"></span>';\r
        },\r
        \r
        _buttonHtml: function() {\r