From 4193d56c23c2b47f0b20724225ff65a4eca33aac Mon Sep 17 00:00:00 2001
From: Aleksandr Klimenkov <alx.devel@gmail.com>
Date: Sun, 31 Aug 2014 17:01:54 +0400
Subject: [PATCH 1/5] trigger change event

---
 jquery.inputNumber.js | 20 ++++++++++++--------
 1 file changed, 12 insertions(+), 8 deletions(-)

diff --git a/jquery.inputNumber.js b/jquery.inputNumber.js
index 031d3bb..b01593e 100644
--- a/jquery.inputNumber.js
+++ b/jquery.inputNumber.js
@@ -12,6 +12,10 @@
         this.init();
     };
 
+    var setValue = function(element, value) {
+        $(element).val(value).change();
+    };
+
     InputNumber.prototype = {
 
         el: null, // input element
@@ -31,7 +35,7 @@
 
         init: function() {
             var opts = this.options;
-            
+
             this.$el.wrap($('<div />', {'class':opts.wrapClass}));
             this.$el.after(
                 $('<a />', {'class':opts.upClass, 'title':opts.upTitle}),
@@ -78,14 +82,14 @@
 
                     if (! opts.negative) {
                         if (value >= 0) {
-                            e.currentTarget.value = value;
+                            setValue(e.currentTarget, value);
                         }
                     } else if (! opts.positive) {
                         if (value <= 0) {
-                            e.currentTarget.value = value;
+                            setValue(e.currentTarget, value);
                         }
                     } else {
-                        e.currentTarget.value = value;
+                        setValue(e.currentTarget, value);
                     }
                 });
         },
@@ -109,10 +113,10 @@
 
             if (! this.options.positive) {
                 if (value <= 0) {
-                    this.el.value = value;
+                    setValue(this.el, value);
                 }
             } else {
-                this.el.value = value;
+                setValue(this.el, value);
             }
         },
 
@@ -121,10 +125,10 @@
 
             if (! this.options.negative) {
                 if (value >= 0) {
-                    this.el.value = value;
+                    setValue(this.el, value);
                 }
             } else {
-                this.el.value = value;
+                setValue(this.el, value);
             }
         }
 

From af224544c6d795992d817be4150cadbae4c0ac8e Mon Sep 17 00:00:00 2001
From: Aleksandr Klimenkov <alx.devel@gmail.com>
Date: Fri, 24 Oct 2014 02:23:37 +0400
Subject: [PATCH 2/5] change markup

---
 inputNumber.css       | 12 ++++++------
 jquery.inputNumber.js |  8 ++++----
 2 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/inputNumber.css b/inputNumber.css
index 01cbede..5013675 100644
--- a/inputNumber.css
+++ b/inputNumber.css
@@ -5,8 +5,8 @@
     margin-right: 10px;
 }
 
-.ranged-input a.up,
-.ranged-input a.down {
+.ranged-input button.up,
+.ranged-input button.down {
     right: 0;
     display: block;
     position: absolute;
@@ -19,20 +19,20 @@
     border: 1px solid #cccccc;
 }
 
-.ranged-input a.up {
+.ranged-input button.up {
     top: 0;
 }
 
-.ranged-input a.up:hover {
+.ranged-input button.up:hover {
     background-position: -20px 0;
 }
 
-.ranged-input a.down {
+.ranged-input button.down {
     background-position: 0 -9px;
     bottom: 0;
 }
 
-.ranged-input a.down:hover {
+.ranged-input button.down:hover {
     background-position: -20px -9px;
 }
 
diff --git a/jquery.inputNumber.js b/jquery.inputNumber.js
index b01593e..c6258f9 100644
--- a/jquery.inputNumber.js
+++ b/jquery.inputNumber.js
@@ -38,8 +38,8 @@
 
             this.$el.wrap($('<div />', {'class':opts.wrapClass}));
             this.$el.after(
-                $('<a />', {'class':opts.upClass, 'title':opts.upTitle}),
-                $('<a />', {'class':opts.downClass, 'title':opts.downTitle})
+                $('<button />', {'class':opts.upClass, 'title':opts.upTitle}),
+                $('<button />', {'class':opts.downClass, 'title':opts.downTitle})
             );
             this.$wrap = this.$el.parent('.'+opts.wrapClass);
 
@@ -52,11 +52,11 @@
                 self = this;
 
             this.$wrap
-                .delegate('a.' + opts.downClass, 'click', function(e) {
+                .delegate('button.' + opts.downClass, 'click', function(e) {
                     self.down();
                     e.preventDefault();
                 })
-                .delegate('a.' + opts.upClass, 'click', function(e) {
+                .delegate('button.' + opts.upClass, 'click', function(e) {
                     self.up();
                     e.preventDefault();
                 });

From 2817c8ab65d8880f93787c8a55959ebeffba9315 Mon Sep 17 00:00:00 2001
From: Aleksandr Klimenkov <alx.devel@gmail.com>
Date: Fri, 21 Nov 2014 17:35:22 +0500
Subject: [PATCH 3/5] value filtration

---
 jquery.inputNumber.js | 91 ++++++++++++++++++++++++++-----------------
 1 file changed, 56 insertions(+), 35 deletions(-)

diff --git a/jquery.inputNumber.js b/jquery.inputNumber.js
index c6258f9..1b3667f 100644
--- a/jquery.inputNumber.js
+++ b/jquery.inputNumber.js
@@ -12,8 +12,27 @@
         this.init();
     };
 
-    var setValue = function(element, value) {
-        $(element).val(value).change();
+    var initDisabledState = function($element, options) {
+        var value = parseInt($element.val());
+        if (isNaN(value)) {
+            value = 0;
+        }
+        $element.parent().find('.' + options.upClass).attr('disabled', !isNaN(options.max) && value >= options.max);
+        $element.parent().find('.' + options.downClass).attr('disabled', !isNaN(options.min) && value <= options.min);
+    };
+
+    var filterValue = function($element, options) {
+        var value = $element.val();
+
+        if (!isNaN(options.min) && value < options.min) {
+            $element.val(options.min);
+        }
+
+        if (!isNaN(options.max) && value > options.max) {
+            $element.val(options.max);
+        }
+
+        initDisabledState($element, options);
     };
 
     InputNumber.prototype = {
@@ -24,18 +43,28 @@
 
         options: null,
         defaults: {
-            'negative': true,
-            'positive': true,
             'wrapClass': 'ranged-input',
             'upClass': 'up',
             'upTitle': 'incrase',
             'downClass': 'down',
-            'downTitle': 'decrace'
+            'downTitle': 'decrace',
+            'min': NaN,
+            'max': NaN
         },
 
         init: function() {
             var opts = this.options;
 
+            var minValue = parseInt(this.el.min);
+            if (!isNaN(minValue)) {
+                this.options.min = minValue;
+            }
+
+            var maxValue = parseInt(this.el.max);
+            if (!isNaN(maxValue)) {
+                this.options.max = maxValue;
+            }
+
             this.$el.wrap($('<div />', {'class':opts.wrapClass}));
             this.$el.after(
                 $('<button />', {'class':opts.upClass, 'title':opts.upTitle}),
@@ -44,6 +73,7 @@
             this.$wrap = this.$el.parent('.'+opts.wrapClass);
 
             this.bindEvents();
+            initDisabledState(this.$el, opts);
         },
 
         bindEvents:function() {
@@ -64,10 +94,10 @@
             $el
                 .on('change', function(e) {
                     if (e.currentTarget.value === '') e.currentTarget.value = 0;
+                    filterValue($el, opts);
                 })
                 .on('keypress', function(e) {
                     var keyCode = window.event ? e.keyCode : e.which;
-
                     //codes for 0-9
                     if (keyCode < 48 || keyCode > 57) {
                         //codes for backspace, delete, enter
@@ -77,20 +107,7 @@
                     }
                 })
                 .bind('mousewheel', function(e, delta) {
-                    var defVal = self.defaultElValue(),
-                        value = defVal + delta;
-
-                    if (! opts.negative) {
-                        if (value >= 0) {
-                            setValue(e.currentTarget, value);
-                        }
-                    } else if (! opts.positive) {
-                        if (value <= 0) {
-                            setValue(e.currentTarget, value);
-                        }
-                    } else {
-                        setValue(e.currentTarget, value);
-                    }
+                    self.setValue(self.defaultElValue() + delta);
                 });
         },
 
@@ -110,28 +127,32 @@
 
         up: function() {
             var value = this.incrementElValue();
-
-            if (! this.options.positive) {
-                if (value <= 0) {
-                    setValue(this.el, value);
-                }
-            } else {
-                setValue(this.el, value);
-            }
+            this.setValue(value);
         },
 
         down: function() {
             var value = this.decrementElValue();
+            this.setValue(value);
+        },
+        setValue: function(value) {
+            value = parseInt(value);
 
-            if (! this.options.negative) {
-                if (value >= 0) {
-                    setValue(this.el, value);
-                }
-            } else {
-                setValue(this.el, value);
+            if (isNaN(value)) {
+                return false;
             }
-        }
 
+            if (!isNaN(this.options.min) && value < this.options.min) {
+                return false;
+            }
+
+            if (!isNaN(this.options.max) && value > this.options.max) {
+                return false;
+            }
+
+            this.$el.val(value).change();
+
+            return true;
+        }
     };
 
     $.fn.inputNumber = function(options) {

From 6f9acbf1e114471bd32a789a8bd8ba718833d530 Mon Sep 17 00:00:00 2001
From: Aleksandr Klimenkov <alx.devel@gmail.com>
Date: Mon, 24 Nov 2014 02:56:05 +0500
Subject: [PATCH 4/5] remove method

---
 jquery.inputNumber.js | 20 +++++++++++++++-----
 1 file changed, 15 insertions(+), 5 deletions(-)

diff --git a/jquery.inputNumber.js b/jquery.inputNumber.js
index 1b3667f..47cb19d 100644
--- a/jquery.inputNumber.js
+++ b/jquery.inputNumber.js
@@ -13,6 +13,7 @@
     };
 
     var initDisabledState = function($element, options) {
+        options = options || $element.data('inputNumber').options;
         var value = parseInt($element.val());
         if (isNaN(value)) {
             value = 0;
@@ -21,9 +22,9 @@
         $element.parent().find('.' + options.downClass).attr('disabled', !isNaN(options.min) && value <= options.min);
     };
 
-    var filterValue = function($element, options) {
+    var filterValue = function($element) {
         var value = $element.val();
-
+        var options = $element.data('inputNumber').options;
         if (!isNaN(options.min) && value < options.min) {
             $element.val(options.min);
         }
@@ -32,7 +33,7 @@
             $element.val(options.max);
         }
 
-        initDisabledState($element, options);
+        initDisabledState($element);
     };
 
     InputNumber.prototype = {
@@ -73,7 +74,7 @@
             this.$wrap = this.$el.parent('.'+opts.wrapClass);
 
             this.bindEvents();
-            initDisabledState(this.$el, opts);
+            initDisabledState(this.$el, this.options);
         },
 
         bindEvents:function() {
@@ -94,7 +95,7 @@
             $el
                 .on('change', function(e) {
                     if (e.currentTarget.value === '') e.currentTarget.value = 0;
-                    filterValue($el, opts);
+                    filterValue($el);
                 })
                 .on('keypress', function(e) {
                     var keyCode = window.event ? e.keyCode : e.which;
@@ -152,6 +153,15 @@
             this.$el.val(value).change();
 
             return true;
+        },
+        remove: function() {
+            this.$el.removeData('inputNumber');
+            var container = this.$el.closest('.' + this.options.wrapClass);
+
+            container.removeClass(this.options.wrapClass)
+                .find('.' + this.options.upClass)
+                .add(container.find('.' + this.options.downClass))
+                .remove();
         }
     };
 

From 391d8442bd90f63613ae7fb100420156b85f2a49 Mon Sep 17 00:00:00 2001
From: Vitaly <orloff.v@gmail.com>
Date: Tue, 31 Mar 2015 19:14:49 +0300
Subject: [PATCH 5/5] Fix undelegate events

---
 jquery.inputNumber.js | 17 +++++++++--------
 1 file changed, 9 insertions(+), 8 deletions(-)

diff --git a/jquery.inputNumber.js b/jquery.inputNumber.js
index 47cb19d..200d800 100644
--- a/jquery.inputNumber.js
+++ b/jquery.inputNumber.js
@@ -83,21 +83,21 @@
                 self = this;
 
             this.$wrap
-                .delegate('button.' + opts.downClass, 'click', function(e) {
+                .delegate('button.' + opts.downClass, 'click.inputNumber', function(e) {
                     self.down();
                     e.preventDefault();
                 })
-                .delegate('button.' + opts.upClass, 'click', function(e) {
+                .delegate('button.' + opts.upClass, 'click.inputNumber', function(e) {
                     self.up();
                     e.preventDefault();
                 });
 
             $el
-                .on('change', function(e) {
+                .on('change.inputNumber', function(e) {
                     if (e.currentTarget.value === '') e.currentTarget.value = 0;
                     filterValue($el);
                 })
-                .on('keypress', function(e) {
+                .on('keypress.inputNumber', function(e) {
                     var keyCode = window.event ? e.keyCode : e.which;
                     //codes for 0-9
                     if (keyCode < 48 || keyCode > 57) {
@@ -107,7 +107,7 @@
                         }
                     }
                 })
-                .bind('mousewheel', function(e, delta) {
+                .bind('mousewheel.inputNumber', function(e, delta) {
                     self.setValue(self.defaultElValue() + delta);
                 });
         },
@@ -156,11 +156,12 @@
         },
         remove: function() {
             this.$el.removeData('inputNumber');
-            var container = this.$el.closest('.' + this.options.wrapClass);
+            this.$wrap.undelegate('.inputNumber');
+            this.$el.off('.inputNumber');
 
-            container.removeClass(this.options.wrapClass)
+            this.$wrap.removeClass(this.options.wrapClass)
                 .find('.' + this.options.upClass)
-                .add(container.find('.' + this.options.downClass))
+                .add(this.$wrap.find('.' + this.options.downClass))
                 .remove();
         }
     };