« get me outta code hell

'Clear past / up to this track' queue menu options - mtui - Music Text User Interface - user-friendly command line music player
about summary refs log tree commit diff
diff options
context:
space:
mode:
authorFlorrie <towerofnix@gmail.com>2019-09-10 12:23:41 -0300
committerFlorrie <towerofnix@gmail.com>2019-09-10 12:23:41 -0300
commit47bd859d1a02683395bbf941d7dc0375d7ef8334 (patch)
tree484c6fdc82af9b14a2ff1e6894d2ad3b060aa3c6
parent366b6e90f1d237a8e1e0d92224f65990157d0cd6 (diff)
'Clear past / up to this track' queue menu options
-rw-r--r--backend.js31
-rw-r--r--todo.txt13
-rw-r--r--ui.js22
3 files changed, 66 insertions, 0 deletions
diff --git a/backend.js b/backend.js
index 0ddd15b..57910e9 100644
--- a/backend.js
+++ b/backend.js
@@ -316,6 +316,37 @@ class Backend extends EventEmitter {
     return focusItem
   }
 
+  clearQueuePast(track) {
+    const { items } = this.queueGrouplike
+    const index = items.indexOf(track) + 1
+
+    if (index < 0) {
+      return
+    } else if (index < items.indexOf(this.playingTrack)) {
+      items.splice(index, items.length - index, this.playingTrack)
+    } else {
+      items.splice(index)
+    }
+
+    this.emitQueueUpdated()
+  }
+
+  clearQueueUpTo(track) {
+    const { items } = this.queueGrouplike
+    const endIndex = items.indexOf(track)
+    const startIndex = (this.playingTrack ? items.indexOf(this.playingTrack) + 1 : 0)
+
+    if (endIndex < 0) {
+      return
+    } else if (endIndex < startIndex) {
+      return
+    } else {
+      items.splice(startIndex, endIndex - startIndex)
+    }
+
+    this.emitQueueUpdated()
+  }
+
   playSooner(item) {
     this.distributeQueue(item, {
       how: 'randomly',
diff --git a/todo.txt b/todo.txt
index 216a488..14c2dd1 100644
--- a/todo.txt
+++ b/todo.txt
@@ -288,3 +288,16 @@ TODO: When you queue a song which is already before the current song / insert
       index in the queue, it ends up being placed one index closer to the end
       than intended. Fix this!
       (Done!)
+
+TODO: A "clear after this track" option. Maybe also a corresponding "clear
+      up to this track" option?
+      (Done!)
+
+TODO: Maybe also a "pause after this track" option. Worth considering!
+
+TODO: Pay closer attention to the ordering of queue items' context menu...
+      I like how it is right now for accessibility to specific menu items
+      (play sooner/later immediately under the start position, remove from
+      queue one "above" the start pos = at the bottom) -- but it might make
+      more logical sense to group 'play sooner', 'play next', and 'remove
+      from queue' all into one section.
diff --git a/ui.js b/ui.js
index 8d13f57..fb73061 100644
--- a/ui.js
+++ b/ui.js
@@ -565,6 +565,24 @@ class AppElement extends FocusElement {
     this.queueListingElement.selectAndShow(item)
   }
 
+  clearQueuePast(item) {
+    if (!this.config.canControlQueue) {
+      return
+    }
+
+    this.backend.clearQueuePast(item)
+    this.queueListingElement.selectAndShow(item)
+  }
+
+  clearQueueUpTo(item) {
+    if (!this.config.canControlQueue) {
+      return
+    }
+
+    this.backend.clearQueueUpTo(item)
+    this.queueListingElement.selectAndShow(item)
+  }
+
   deselectAll() {
     this.markGrouplike.items.splice(0)
   }
@@ -599,6 +617,10 @@ class AppElement extends FocusElement {
         {divider: true},
         canControlQueue && {label: 'Play later', action: () => this.playLater(item)},
         canControlQueue && {label: 'Play sooner', action: () => this.playSooner(item)},
+        {divider: true},
+        canControlQueue && {label: 'Clear past this track', action: () => this.clearQueuePast(item)},
+        canControlQueue && {label: 'Clear up to this track', action: () => this.clearQueueUpTo(item)},
+        {divider: true},
         canControlQueue && {label: 'Remove from queue', action: () => this.unqueue(item)}
       ]
     } else {