IonDen / ion.rangeSlider

jQuery only range slider
http://ionden.com/a/plugins/ion.rangeSlider/en.html
MIT License
2.55k stars 505 forks source link

"from" does not default to "min" #9

Closed flip111 closed 11 years ago

flip111 commented 11 years ago

Issue: "from" does not default to "min" (default to 10 instead)

Browser: Firefox 23.0.1 OS: Win7 Pro 64 bit IonRangeSlider: 1.6.3

Test Case:

<html>
    <head>
        <meta http-equiv="content-type" content="text/html; charset=utf-8" />
        <title>Slider example</title>
        <script type="text/javascript" language="javascript" src="js/jquery-2.0.2.min.js"></script>
        <script type="text/javascript" language="javascript" src="slider/ion.rangeSlider.min.js"></script>
        <link href="slider/normalize.min.css" rel="stylesheet" type="text/css" />
        <link href="slider/ion.rangeSlider.css" rel="stylesheet" type="text/css" />
        <link href="slider/ion.rangeSlider.skinSimple.css" rel="stylesheet" type="text/css" />
    </head>
  <body>
    <input type="text" id="test" name="rangeName" value="" />
        <script type="text/javascript" charset="utf-8">
            $('#test').ionRangeSlider({
                min: 0,
                max: 100,
                type: 'double',
                step: 1,
                hasGrid: false,
                hideText: false
            });
        </script>
  </body>
</html>

skinSimple was edited to look for the image in the same directory.

IonDen commented 11 years ago

http://ionden.com/a/plugins/ion.rangeSlider/static/js/ion-rangeSlider/ion.rangeSlider.js Try this, it also fixes this bug

flip111 commented 11 years ago

This bug is still present with ion range slider 1.7.0 build 120

IonDen commented 11 years ago

http://ionden.com/a/plugins/ion.rangeSlider/static/js/ion-rangeSlider/ion.rangeSlider.js Try this, please.

flip111 commented 11 years ago

This bug is still present with ion range slider 1.7.0 build 122 Tested on Firefox 23.0.1 and Google Chrome 29.0.1547.57

IonDen commented 11 years ago

http://ionden.com/a/plugins/ion.rangeSlider/static/js/ion-rangeSlider/ion.rangeSlider.js Hey, now it is fixed :) Build 123

flip111 commented 11 years ago

Yes i confirm, it's fixed.

By the way i choose this slider because it it is small, looks really nice, it has nice features like text above sliders and grid. By comparison: Ion Range Slider: 6KB + sprite PNG 1KB jQuery UI: 23 KB (when only selecting slider and dependencies at download) noUiSlider: 6kb (but doesn't look as nice) JSlider: 20 KB + lots of dependencies jQRangeSlider: 39Kb (and larger, depending on version)

IonDen commented 11 years ago

Tnx for using.