r3014 committed - refactor
Revision: 3014
Author: rdworth
Date: Fri Jul 31 05:47:55 2009
Log: refactor
http://code.google.com/p/jquery-ui/source/detail?r=3014
Modified:
/branches/dev/spinner/ui/ui.spinner.js
=======================================
--- /branches/dev/spinner/ui/ui.spinner.js Thu Jul 30 20:52:07 2009
+++ /branches/dev/spinner/ui/ui.spinner.js Fri Jul 31 05:47:55 2009
@@ -73,10 +73,10 @@
var self = this,
options = self.options;
- var widget = self.element
+ var uiSpinner = self.element
.addClass('ui-spinner-box')
.attr('autocomplete', 'off') // switch off autocomplete in opera
- .wrap(self._widgetHtml())
+ .wrap(self._uiSpinnerHtml())
.parent()
// add buttons
.append(self._buttonHtml())
@@ -99,8 +99,8 @@
// 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');
+ if (!$.support.opacity && uiSpinner.css('display') == 'inline-block' &&
$.browser.version < 8) {
+ uiSpinner.css('display', 'inline');
}
// element bindings
@@ -109,7 +109,7 @@
// - this should aid targetted customisations if a page contains
multiple instances
.attr('id', function(){
if (this.id) {
- widget.attr('id', 'ui-spinner-'+ this.id);
+ uiSpinner.attr('id', 'ui-spinner-'+ this.id);
}
})
.bind('keydown'+namespace, function(event) {
@@ -125,12 +125,12 @@
self.element.blur();
return false;
}
- widget.addClass(active);
+ uiSpinner.addClass(active);
self.focused = true;
})
.bind('blur'+namespace, function(event) {
if (!self.hovered) {
- widget.removeClass(active);
+ uiSpinner.removeClass(active);
}
self.focused = false;
self._cleanUp();
@@ -147,14 +147,14 @@
}
// button bindings
- this.buttons = widget.find('.ui-spinner-button')
+ this.buttons = uiSpinner.find('.ui-spinner-button')
.bind('mousedown', function(event) {
self._start(event);
self._repeat(null, $(this).hasClass('ui-spinner-up') ? 1 : -1, event);
if (!self.options.disabled && !self.options.readOnly) {
$(this).addClass(active);
- widget.addClass(active);
+ uiSpinner.addClass(active);
}
})
.bind('mouseup', function(event) {
@@ -188,9 +188,9 @@
}
});
- self.widget = widget;
+ self.uiSpinner = uiSpinner;
},
- _widgetHtml: function() {
+ _uiSpinnerHtml: function() {
return '<div role="spinbutton" class="ui-spinner ui-state-default
ui-widget ui-widget-content ui-corner-all ' +
(this.options.spinnerClass || '') +
' ui-spinner-' + this.options.dir +
@@ -440,8 +440,8 @@
.removeData('spinner')
.unbind(namespace);
- if (this.widget) {
- this.widget.replaceWith(this.element);
+ if (this.uiSpinner) {
+ this.uiSpinner.replaceWith(this.element);
}
},
enable: function() {
@@ -450,7 +450,7 @@
.siblings()
.removeAttr('disabled')
.parent()
- .removeClass('ui-state-disabled');
+ .removeClass('ui-spinner-disabled ui-state-disabled');
this.options.disabled = false;
},
disable: function() {
@@ -459,7 +459,7 @@
.siblings()
.attr('disabled', true)
.parent()
- .addClass('ui-state-disabled');
+ .addClass('ui-spinner-disabled ui-state-disabled');
this.options.disabled = true;
},
value: function(newVal) {