Merge pull request #67 from ccatterina/track-position-slider-improvem…

…ents fix: set position instead of use seek function
ccatterina · Mar 16, 2024 · b612552 · b612552
2 parents 04cc966 + bbf1a93
commit b612552
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 7 deletions.
diff --git a/src/contents/ui/Player.qml b/src/contents/ui/Player.qml
@@ -67,8 +67,8 @@ QtObject {
         mpris2Model.currentPlayer?.PlayPause();
     }
 
-    function seek(offset) {
-        mpris2Model.currentPlayer?.Seek(offset);
+    function setPosition(position) {
+        mpris2Model.currentPlayer.position = position;
     }
 
     function next() {

diff --git a/src/contents/ui/TrackPositionSlider.qml b/src/contents/ui/TrackPositionSlider.qml
@@ -47,13 +47,12 @@ Item {
                 if (pressed) {
                     return
                 }
-                changingPosition = true
 
+                changingPosition = true
                 const targetPosition = timeTrackSlider.value * container.songLength
                 if (targetPosition != container.songPosition) {
-                    container.requireChangePosition(Math.round(targetPosition - container.songPosition))
+                    container.requireChangePosition(targetPosition)
                 }
-
                 changingPosition = false
             }
         }

diff --git a/src/contents/ui/main.qml b/src/contents/ui/main.qml
@@ -142,8 +142,8 @@ PlasmoidItem {
                 songLength: player.songLength
                 playing: player.playbackStatus === Mpris.PlaybackStatus.Playing
                 enableChangePosition: player.canSeek
-                onRequireChangePosition: (delta) => {
-                    player.seek(delta)
+                onRequireChangePosition: (position) => {
+                    player.setPosition(position)
                 }
                 onRequireUpdatePosition: () => {
                     player.updatePosition()