Update simple-slider.js

anasAsh · anasAsh · commit c08e15e1f616 · 2013-04-02T11:53:29.000+03:00
Jquery hide() uses display:none css, which prevents submitting the input value. anyway, i think developers should use hidden input type with the slider.
diff --git a/js/simple-slider.js b/js/simple-slider.js
@@ -28,7 +28,7 @@ var __slice = [].slice,
       if (this.settings.theme) {
         this.settings.classSuffix = "-" + this.settings.theme;
       }
-      this.input.hide();
+      this.input.css('visibility', 'hidden');
       this.slider = $("<div>").addClass("slider" + (this.settings.classSuffix || "")).css({
         position: "relative",
         userSelect: "none",

Original file line number	Diff line number	Diff line change
`@@ -28,7 +28,7 @@ var __slice = [].slice,`
`28`	`28`	`if (this.settings.theme) {`
`29`	`29`	`this.settings.classSuffix = "-" + this.settings.theme;`
`30`	`30`	`}`
`31`		`- this.input.hide();`
	`31`	`+ this.input.css('visibility', 'hidden');`
`32`	`32`	`this.slider = $("<div>").addClass("slider" + (this.settings.classSuffix \|\| "")).css({`
`33`	`33`	`position: "relative",`
`34`	`34`	`userSelect: "none",`