* Show no. of items to be deleted on prompts

PikachuEXE · Jun 11, 2024 · e6f6b75 · e6f6b75
1 parent 74d98be
commit e6f6b75
Show file tree

Hide file tree

Showing 3 changed files with 27 additions and 4 deletions.
diff --git a/src/renderer/components/playlist-info/playlist-info.js b/src/renderer/components/playlist-info/playlist-info.js
@@ -166,6 +166,20 @@ export default defineComponent({
         this.$t('Cancel')
       ]
     },
+    removeVideosOnWatchPromptLabelText() {
+      return this.$tc(
+        'User Playlists.Are you sure you want to remove {playlistItemCount} watched videos from this playlist? This cannot be undone',
+        this.userPlaylistWatchedVideoCount,
+        { playlistItemCount: this.userPlaylistWatchedVideoCount },
+      )
+    },
+    removeDuplicateVideosPromptLabelText() {
+      return this.$tc(
+        'User Playlists.Are you sure you want to remove {playlistItemCount} duplicate videos from this playlist? This cannot be undone',
+        this.userPlaylistDuplicateItemCount,
+        { playlistItemCount: this.userPlaylistDuplicateItemCount },
+      )
+    },
 
     firstVideoIdExists() {
       return this.firstVideoId !== ''
@@ -220,6 +234,15 @@ export default defineComponent({
         return typeof historyCacheById[video.videoId] !== 'undefined'
       })
     },
+    // `userPlaylistAnyVideoWatched` is faster than this & this is only needed when prompt shown
+    userPlaylistWatchedVideoCount() {
+      if (!this.isUserPlaylist) { return false }
+
+      const historyCacheById = this.$store.getters.getHistoryCacheById
+      return this.selectedUserPlaylist.videos.reduce((count, video) => {
+        return typeof historyCacheById[video.videoId] !== 'undefined' ? count + 1 : count
+      }, 0)
+    },
 
     userPlaylistUniqueVideoIds() {
       if (!this.isUserPlaylist) { return new Set() }

diff --git a/src/renderer/components/playlist-info/playlist-info.vue b/src/renderer/components/playlist-info/playlist-info.vue
@@ -210,15 +210,15 @@
       />
       <ft-prompt
         v-if="showRemoveVideosOnWatchPrompt"
-        :label="$t('User Playlists.Are you sure you want to remove all watched videos from this playlist? This cannot be undone')"
+        :label="removeVideosOnWatchPromptLabelText"
         :option-names="deletePlaylistPromptNames"
         :option-values="deletePlaylistPromptValues"
         :is-first-option-destructive="true"
         @click="handleRemoveVideosOnWatchPromptAnswer"
       />
       <ft-prompt
         v-if="showRemoveDuplicateVideosPrompt"
-        :label="$t('User Playlists.Are you sure you want to remove duplicate videos from this playlist? This cannot be undone')"
+        :label="removeDuplicateVideosPromptLabelText"
         :option-names="deletePlaylistPromptNames"
         :option-values="deletePlaylistPromptValues"
         :is-first-option-destructive="true"

diff --git a/static/locales/en-US.yaml b/static/locales/en-US.yaml
@@ -189,8 +189,8 @@ User Playlists:
   Remove Watched Videos: Remove Watched Videos
   Enable Quick Bookmark With This Playlist: Enable Quick Bookmark With This Playlist
   Quick Bookmark Enabled: Quick Bookmark Enabled
-  Are you sure you want to remove duplicate videos from this playlist? This cannot be undone: Are you sure you want to remove duplicate videos from this playlist? This cannot be undone.
-  Are you sure you want to remove all watched videos from this playlist? This cannot be undone: Are you sure you want to remove all watched videos from this playlist? This cannot be undone.
+  Are you sure you want to remove {playlistItemCount} duplicate videos from this playlist? This cannot be undone: Are you sure you want to remove 1 duplicate video from this playlist? This cannot be undone. | Are you sure you want to remove {playlistItemCount} duplicate videos from this playlist? This cannot be undone.
+  Are you sure you want to remove {playlistItemCount} watched videos from this playlist? This cannot be undone: Are you sure you want to remove 1 watched video from this playlist? This cannot be undone. | Are you sure you want to remove {playlistItemCount} watched videos from this playlist? This cannot be undone.
   Delete Playlist: Delete Playlist
   Cannot delete the quick bookmark target playlist.: Cannot delete the quick bookmark target playlist.
   Are you sure you want to delete this playlist? This cannot be undone: Are you sure you want to delete this playlist? This cannot be undone.