openstack/horizon
Code Issues Proposed changes

Merge "fix the bug of checkSpinnerValue"

Browse Source
This commit is contained in:
Zuul 2018-11-21 22:01:02 +00:00 committed by Gerrit Code Review
commit f535687c9f
1 changed files with 18 additions and 15 deletions

33
horizon/static/horizon/js/horizon.forms.js

@ -365,25 +365,28 @@ horizon.forms.getSpinnerValue = function(val, defaultVal) {
return isNaN(val) ? defaultVal : val; return isNaN(val) ? defaultVal : val;
}; };
horizon.forms.checkSpinnerValue = function($input) { horizon.forms.checkSpinnerValue = function($inputs) {
var val = $input.attr('value'); $inputs.each(function (index, input) {
var max = horizon.forms.getSpinnerValue($input.attr('max'), Number.MAX_SAFE_INTEGER); var $input = $(input);
var min = horizon.forms.getSpinnerValue($input.attr('min'), 0); var val = $input.attr('value');
var max = horizon.forms.getSpinnerValue($input.attr('max'), Number.MAX_SAFE_INTEGER);
var min = horizon.forms.getSpinnerValue($input.attr('min'), 0);
var $parent = $input.parents('.themable-spinner'); var $parent = $input.parents('.themable-spinner');
var $up = $parent.find('.spinner-up'); var $up = $parent.find('.spinner-up');
var $down = $parent.find('.spinner-down'); var $down = $parent.find('.spinner-down');
$parent.find('.themable-spinner-btn').removeAttr('disabled'); $parent.find('.themable-spinner-btn').removeAttr('disabled');
if (val <= min) { if (val <= min) {
// Disable if we've hit the min // Disable if we've hit the min
$down.attr('disabled', true); $down.attr('disabled', true);
} else if (val >= max) { } else if (val >= max) {
// Disable if we've hit the max // Disable if we've hit the max
$up.attr('disabled', true); $up.attr('disabled', true);
} }
});
}; };
horizon.forms.init_themable_spinner = function ($elem) { horizon.forms.init_themable_spinner = function ($elem) {