mirror of
https://github.com/jellyfin/jellyfin-web
synced 2025-03-30 19:56:21 +00:00
Merge pull request #5601 from thornbill/fix-marker-updating
This commit is contained in:
commit
167515dbf0
2 changed files with 18 additions and 22 deletions
|
@ -1835,21 +1835,11 @@ export default function (view) {
|
|||
};
|
||||
|
||||
nowPlayingPositionSlider.getMarkerInfo = function () {
|
||||
const markers = [];
|
||||
|
||||
const item = currentItem;
|
||||
|
||||
// use markers based on chapters
|
||||
if (item?.Chapters?.length) {
|
||||
item.Chapters.forEach(currentChapter => {
|
||||
markers.push({
|
||||
return currentItem?.Chapters?.map(currentChapter => ({
|
||||
name: currentChapter.Name,
|
||||
progress: currentChapter.StartPositionTicks / item.RunTimeTicks
|
||||
});
|
||||
});
|
||||
}
|
||||
|
||||
return markers;
|
||||
progress: currentChapter.StartPositionTicks / currentItem.RunTimeTicks
|
||||
})) || [];
|
||||
};
|
||||
|
||||
view.querySelector('.btnPreviousTrack').addEventListener('click', function () {
|
||||
|
|
|
@ -1,3 +1,5 @@
|
|||
import isEqual from 'lodash-es/isEqual';
|
||||
|
||||
import browser from '../../scripts/browser';
|
||||
import dom from '../../scripts/dom';
|
||||
import layoutManager from '../../components/layoutManager';
|
||||
|
@ -203,8 +205,11 @@ function setMarker(range, valueMarker, marker, valueProgress) {
|
|||
}
|
||||
|
||||
function updateMarkers(range, currentValue) {
|
||||
if (range.getMarkerInfo && !range.markerInfo) {
|
||||
range.markerInfo = range.getMarkerInfo();
|
||||
if (range.getMarkerInfo) {
|
||||
const newMarkerInfo = range.getMarkerInfo();
|
||||
|
||||
if (!range.markerInfo || !isEqual(range.markerInfo, newMarkerInfo)) {
|
||||
range.markerInfo = newMarkerInfo;
|
||||
|
||||
let markersHtml = '';
|
||||
range.markerInfo.forEach(() => {
|
||||
|
@ -214,6 +219,7 @@ function updateMarkers(range, currentValue) {
|
|||
|
||||
range.markerElements = range.markerContainerElement.querySelectorAll('.sliderMarker');
|
||||
}
|
||||
}
|
||||
|
||||
if (range.markerInfo?.length && range.markerElements?.length) {
|
||||
for (let i = 0, length = range.markerElements.length; i < length; i++) {
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue