diff --git a/IPython/html/static/notebook/js/widgets/widget_float_range.js b/IPython/html/static/notebook/js/widgets/widget_float_range.js index 10110eb..22d97d6 100644 --- a/IPython/html/static/notebook/js/widgets/widget_float_range.js +++ b/IPython/html/static/notebook/js/widgets/widget_float_range.js @@ -23,8 +23,7 @@ define(["notebook/js/widgets/widget"], function(widget_manager){ // Called when view is rendered. render : function(){ this.$el - .addClass('widget-hbox-single') - .html(''); + .addClass('widget-hbox-single'); this.$label = $('
') .appendTo(this.$el) .addClass('widget-hlabel') @@ -51,10 +50,11 @@ define(["notebook/js/widgets/widget"], function(widget_manager){ // changed by another view or by a state update from the back-end. if (options === undefined || options.updated_view != this) { - // Slider related keys. - var _keys = ['step', 'max', 'min', 'disabled']; - for (var index in _keys) { - var key = _keys[index]; + // JQuery slider option keys. These keys happen to have a + // one-to-one mapping with the corrosponding keys of the model. + var jquery_slider_keys = ['step', 'max', 'min', 'disabled']; + for (var index in jquery_slider_keys) { + var key = jquery_slider_keys[index]; if (this.model.get(key) !== undefined) { this.$slider.slider("option", key, this.model.get(key)); } @@ -129,8 +129,7 @@ define(["notebook/js/widgets/widget"], function(widget_manager){ // Called when view is rendered. render : function(){ this.$el - .addClass('widget-hbox-single') - .html(''); + .addClass('widget-hbox-single'); this.$label = $('
') .appendTo(this.$el) .addClass('widget-hlabel') @@ -226,8 +225,7 @@ define(["notebook/js/widgets/widget"], function(widget_manager){ // Called when view is rendered. render : function(){ this.$el - .addClass('widget-hbox-single') - .html(''); + .addClass('widget-hbox-single'); this.$label = $('
') .appendTo(this.$el) .addClass('widget-hlabel') diff --git a/IPython/html/static/notebook/js/widgets/widget_int_range.js b/IPython/html/static/notebook/js/widgets/widget_int_range.js index b53552c..76ede5f 100644 --- a/IPython/html/static/notebook/js/widgets/widget_int_range.js +++ b/IPython/html/static/notebook/js/widgets/widget_int_range.js @@ -23,8 +23,7 @@ define(["notebook/js/widgets/widget"], function(widget_manager){ // Called when view is rendered. render : function(){ this.$el - .addClass('widget-hbox-single') - .html(''); + .addClass('widget-hbox-single'); this.$label = $('
') .appendTo(this.$el) .addClass('widget-hlabel') @@ -50,10 +49,11 @@ define(["notebook/js/widgets/widget"], function(widget_manager){ // Called when the model is changed. The model may have been // changed by another view or by a state update from the back-end. if (options === undefined || options.updated_view != this) { - // Slider related keys. - var _keys = ['step', 'max', 'min', 'disabled']; - for (var index in _keys) { - var key = _keys[index]; + // JQuery slider option keys. These keys happen to have a + // one-to-one mapping with the corrosponding keys of the model. + var jquery_slider_keys = ['step', 'max', 'min', 'disabled']; + for (var index in jquery_slider_keys) { + var key = jquery_slider_keys[index]; if (this.model.get(key) !== undefined) { this.$slider.slider("option", key, this.model.get(key)); } @@ -127,8 +127,7 @@ define(["notebook/js/widgets/widget"], function(widget_manager){ // Called when view is rendered. render : function(){ this.$el - .addClass('widget-hbox-single') - .html(''); + .addClass('widget-hbox-single'); this.$label = $('
') .appendTo(this.$el) .addClass('widget-hlabel')