r2669 - spinner: moved _draw function below _init.
Author: pazu2k@gmail.com
Date: Sun Jun 7 08:55:20 2009
New Revision: 2669
Modified:
branches/dev/spinner/ui/ui.spinner.js
Log:
spinner: moved _draw function below _init.
Modified: branches/dev/spinner/ui/ui.spinner.js
==============================================================================
--- branches/dev/spinner/ui/ui.spinner.js (original)
+++ branches/dev/spinner/ui/ui.spinner.js Sun Jun 7 08:55:20 2009
@@ -211,7 +211,41 @@
self._extend('initComplete');
},
+
+ _draw: function() {
+ var dir = this.options.dir,
+ spinnerClass = this.options.spinnerClass,
+ spinnerBoxClass = 'ui-spinner-box',
+ widgetClasses = 'ui-spinner ui-widget ui-widget-content ui-corner-all
ui-spinner-' + dir + (spinnerClass ? ' '+ spinnerClass: ''),
+ widget = this.element
+ .addClass(spinnerBoxClass)
+ .attr('autocomplete', 'off') // switch off autocomplete in opera
+ .wrap('<div>')
+ .parent()
+ .addClass(widgetClasses)
+ .append('<a class="ui-spinner-button ui-spinner-up ui-state-default
ui-corner-t'+ dir.substr(-1,1) +'"><span
class="ui-spinner-button-inner"><span class="ui-icon
ui-icon-triangle-1-n">▲</span></span></a>')
+ .append('<a class="ui-spinner-button ui-spinner-down ui-state-default
ui-corner-b'+ dir.substr(-1,1) +'"><span
class="ui-spinner-button-inner"><span class="ui-icon
ui-icon-triangle-1-s">▼</span></span></a>');
+
+ // TODO: need a better way to exclude IE8 without resorting to
$.browser.version
+ // fix inline-block issues for IE. Since IE8 supports inline-block we
need to exclude it.
+ if (!$.support.opacity && widget.css('display') == 'inline-block' &&
$.browser.version < 8) {
+ widget.css('display', 'inline');
+ }
+ // force width if passed through options
+ if (this.options.width)
+ widget.find('.'+spinnerBoxClass).width(this.options.width);
+
+ // Give the spinner casing a unique id only if one exists in original
input
+ // - this should aid targetted customisations if a page contains
multiple instances
+ widget.find('.'+spinnerBoxClass).attr('id', function(){
+ if (this.id) {
+ $(this).parent().attr('id', this.id + '-ui-spinner');
+ }
+ });
+
+ return widget;
+ },
_constrain: function() {
if (this.options.min != null && this._getValue() < this.options.min) {
this._setValue(this.options.min);
@@ -434,40 +468,6 @@
}
$.widget.prototype._setData.call(this, key, value);
- },
- _draw: function() {
- var dir = this.options.dir,
- spinnerClass = this.options.spinnerClass,
- spinnerBoxClass = 'ui-spinner-box',
- widgetClasses = 'ui-spinner ui-widget ui-widget-content ui-corner-all
ui-spinner-' + dir + (spinnerClass ? ' '+ spinnerClass: ''),
- widget = this.element
- .addClass(spinnerBoxClass)
- .attr('autocomplete', 'off') // switch off autocomplete in opera
- .wrap('<div>')
- .parent()
- .addClass(widgetClasses)
- .append('<a class="ui-spinner-button ui-spinner-up ui-state-default
ui-corner-t'+ dir.substr(-1,1) +'"><span
class="ui-spinner-button-inner"><span class="ui-icon
ui-icon-triangle-1-n">▲</span></span></a>')
- .append('<a class="ui-spinner-button ui-spinner-down ui-state-default
ui-corner-b'+ dir.substr(-1,1) +'"><span
class="ui-spinner-button-inner"><span class="ui-icon
ui-icon-triangle-1-s">▼</span></span></a>');
-
- // TODO: need a better way to exclude IE8 without resorting to
$.browser.version
- // fix inline-block issues for IE. Since IE8 supports inline-block we
need to exclude it.
- if (!$.support.opacity && widget.css('display') == 'inline-block' &&
$.browser.version < 8) {
- widget.css('display', 'inline');
- }
-
- // force width if passed through options
- if (this.options.width)
- widget.find('.'+spinnerBoxClass).width(this.options.width);
-
- // Give the spinner casing a unique id only if one exists in original
input
- // - this should aid targetted customisations if a page contains
multiple instances
- widget.find('.'+spinnerBoxClass).attr('id', function(){
- if (this.id) {
- $(this).parent().attr('id', this.id + '-ui-spinner');
- }
- });
-
- return widget;
},
plugins: {},