Skip to content

Commit

Permalink
Fix problem with bigger range values
Browse files Browse the repository at this point in the history
  • Loading branch information
Kevin Glier committed Feb 15, 2022
1 parent a732224 commit d1a5e75
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions src/om-javascript-range-slider.js
Original file line number Diff line number Diff line change
Expand Up @@ -160,7 +160,11 @@ function OmRangeSlider(inputElement, inputValueStyle) {
break;
}

that.setRange([range[0] ?? settings.min, range[1] ?? settings.max]);
const rangeStart = range[0] && range[0] >= settings.min && range[0] <= settings.max
? range[0] : settings.min;
const rangeEnd = range[1] && range[1] >= settings.min && range[1] <= settings.max
? range[1] : settings.max;
that.setRange([rangeStart, rangeEnd]);
refreshButtonPositions();
refreshRangeIndicator();
}
Expand Down Expand Up @@ -201,7 +205,7 @@ function OmRangeSlider(inputElement, inputValueStyle) {
}

function updateValues() {
const rangePx = elementRange.offsetWidth - buttonStart.offsetWidth - buttonEnd.offsetWidth;
const rangePx = elementRange.offsetWidth - buttonStart.offsetWidth - buttonEnd.offsetWidth - 1;
const startPx = parseInt(buttonStart.style.left);
const endPx = parseInt(buttonEnd.style.left) - buttonEnd.offsetWidth;

Expand Down

0 comments on commit d1a5e75

Please sign in to comment.