mirrors/jellyfin-web
1
0
Fork 0
mirror of https://github.com/jellyfin/jellyfin-web synced 2025-03-30 19:56:21 +00:00
Code Activity

Fix slider step

Browse source 
Use the value of the `step` attribute if no keyboard steps are specified.
This commit is contained in:
Dmitry Lyzo 2023-08-23 23:58:07 +03:00
parent b2f3720282
commit 7d27596d6b
1 changed files with 25 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

29
src/elements/emby-slider/emby-slider.js
View file

@ -20,6 +20,27 @@ import '../emby-input/emby-input';
}
}
/**
* Returns normalized slider step.
*
* @param {HTMLInputElement} range slider itself
* @param {number|undefined} step step
* @returns {number} normalized slider step.
*/
function normalizeSliderStep(range, step) {
if (step > 0) {
return step;
}
step = parseFloat(range.step);
if (step > 0) {
return step;
}
return 1;
}
/**
* Returns slider fraction corresponding to client position.
*
@ -35,7 +56,7 @@ import '../emby-input/emby-input';
// Snap to step
const valueRange = range.max - range.min;
if (range.step !== 'any' && valueRange !== 0) {
const step = (range.step || 1) / valueRange;
const step = normalizeSliderStep(range) / valueRange;
fraction = Math.round(fraction / step) * step;
}
@ -54,7 +75,7 @@ import '../emby-input/emby-input';
// Snap to step
if (range.step !== 'any') {
const step = range.step || 1;
const step = normalizeSliderStep(range);
value = Math.round(value / step) * step;
}
@ -377,13 +398,13 @@ import '../emby-input/emby-input';
switch (keyboardnavigation.getKeyName(e)) {
case 'ArrowLeft':
case 'Left':
stepKeyboard(this, -this.keyboardStepDown || -1);
stepKeyboard(this, -normalizeSliderStep(this, this.keyboardStepDown));
e.preventDefault();
e.stopPropagation();
break;
case 'ArrowRight':
case 'Right':
stepKeyboard(this, this.keyboardStepUp || 1);
stepKeyboard(this, normalizeSliderStep(this, this.keyboardStepUp));
e.preventDefault();
e.stopPropagation();
break;