setAllItemsEnabled is called before rangeslider is setUp

call setAllItemsEnabled again after rangeslider is setup
adaptlearning · May 24, 2016 · 4400fd1 · 4400fd1
1 parent e46a6cb
commit 4400fd1
Showing 1 changed file with 11 additions and 3 deletions.
diff --git a/js/adapt-contrib-slider.js b/js/adapt-contrib-slider.js
@@ -46,6 +46,10 @@ define([
                 onSlide: _.bind(this.handleSlide, this)
             });
             this.oldValue = 0;
+
+            if (this._deferEnable) {
+                this.setAllItemsEnabled(true);
+            }
         },
 
         handleSlide: function (position, value) {
@@ -133,9 +137,13 @@ define([
 
         setAllItemsEnabled: function(isEnabled) {
             if (isEnabled) {
-                this.$('.slider-widget').removeClass('disabled');
-                this.$slider.prop('disabled', false);
-                this.$slider.rangeslider('update', true);
+                if (this.$slider) {
+                    this.$('.slider-widget').removeClass('disabled');
+                    this.$slider.prop('disabled', false);
+                    this.$slider.rangeslider('update', true);
+                } else {
+                    this._deferEnable = true; // slider is not yet ready
+                }
             } else {
                 this.$('.slider-widget').addClass('disabled');
                 this.$slider.prop('disabled', true);