From 4dea7746c5576bdab887bd7029ae7310b8b8bed6 Mon Sep 17 00:00:00 2001
From: PikachuEXE <pikachuexe@gmail.com>
Date: Tue, 2 Apr 2024 09:54:19 +0800
Subject: [PATCH] Update icons for add to/remove from quick bookmark button
 (#4825)

* * Update icons for add to/remove from quick bookmark target button

* * Change quick bookmarked indicator to green

* * Stop importing star icon which is now unused
---
 .../components/ft-list-video/ft-list-video.vue     |  2 +-
 .../watch-video-info/watch-video-info.vue          |  2 +-
 src/renderer/main.js                               |  4 ++--
 src/renderer/themes.css                            | 14 +++++++-------
 4 files changed, 11 insertions(+), 11 deletions(-)

diff --git a/src/renderer/components/ft-list-video/ft-list-video.vue b/src/renderer/components/ft-list-video/ft-list-video.vue
index 5500b5feb..391ff3c1f 100644
--- a/src/renderer/components/ft-list-video/ft-list-video.vue
+++ b/src/renderer/components/ft-list-video/ft-list-video.vue
@@ -58,7 +58,7 @@
         <ft-icon-button
           v-if="isQuickBookmarkEnabled && quickBookmarkButtonEnabled"
           :title="quickBookmarkIconText"
-          :icon="['fas', 'star']"
+          :icon="isInQuickBookmarkPlaylist ? ['fas', 'check'] : ['fas', 'clock']"
           class="quickBookmarkVideoIcon"
           :class="{
             bookmarked: isInQuickBookmarkPlaylist,
diff --git a/src/renderer/components/watch-video-info/watch-video-info.vue b/src/renderer/components/watch-video-info/watch-video-info.vue
index 76843dbb4..680adcd21 100644
--- a/src/renderer/components/watch-video-info/watch-video-info.vue
+++ b/src/renderer/components/watch-video-info/watch-video-info.vue
@@ -90,7 +90,7 @@
         <ft-icon-button
           v-if="isQuickBookmarkEnabled"
           :title="quickBookmarkIconText"
-          :icon="['fas', 'star']"
+          :icon="isInQuickBookmarkPlaylist ? ['fas', 'check'] : ['fas', 'clock']"
           class="quickBookmarkVideoIcon"
           :class="{
             bookmarked: isInQuickBookmarkPlaylist,
diff --git a/src/renderer/main.js b/src/renderer/main.js
index 6a5cd9136..d836d9cc5 100644
--- a/src/renderer/main.js
+++ b/src/renderer/main.js
@@ -22,6 +22,7 @@ import {
   faCheck,
   faChevronRight,
   faCircleUser,
+  faClock,
   faClone,
   faComment,
   faCommentDots,
@@ -63,7 +64,6 @@ import {
   faShareAlt,
   faSlidersH,
   faSortDown,
-  faStar,
   faStepBackward,
   faStepForward,
   faSync,
@@ -102,6 +102,7 @@ library.add(
   faCheck,
   faChevronRight,
   faCircleUser,
+  faClock,
   faClone,
   faComment,
   faCommentDots,
@@ -143,7 +144,6 @@ library.add(
   faShareAlt,
   faSlidersH,
   faSortDown,
-  faStar,
   faStepBackward,
   faStepForward,
   faSync,
diff --git a/src/renderer/themes.css b/src/renderer/themes.css
index fd100cc5f..f5e279ae5 100644
--- a/src/renderer/themes.css
+++ b/src/renderer/themes.css
@@ -72,7 +72,7 @@
   --primary-shadow-color: rgba(232, 232, 232, 1);
   --title-color: #3f7ac6;
   --bg-color: #f1f1f1;
-  --favorite-icon-color: #FFD600;
+  --favorite-icon-color: #00CC00;
   --card-bg-color: #FFFFFF;
   --secondary-card-bg-color: #eeeeee;
   --scrollbar-color: #CCCCCC;
@@ -89,7 +89,7 @@
   --tertiary-text-color: #999;
   --title-color: #EEEEEE;
   --bg-color: #212121;
-  --favorite-icon-color: #FFEA00;
+  --favorite-icon-color: #00FF00;
   --card-bg-color: #303030;
   --secondary-card-bg-color: rgba(0, 0, 0, 0.75);
   --scrollbar-color: #414141;
@@ -106,7 +106,7 @@
     --tertiary-text-color: #EEEEEE;
     --title-color: #EEEEEE;
     --bg-color: #000000;
-    --favorite-icon-color: #FFEA00;
+    --favorite-icon-color: #00FF00;
     --card-bg-color: #000000;
     --secondary-card-bg-color: rgba(0, 0, 0, 0.75);
     --scrollbar-color: #515151;
@@ -137,7 +137,7 @@
   --tertiary-text-color: #e5e8f3;
   --title-color: #BD93F9;
   --bg-color: #282A36;
-  --favorite-icon-color: #F1FA8C;
+  --favorite-icon-color: #00FF00;
   --card-bg-color: #33353F;
   --secondary-card-bg-color: #282A36;
   --scrollbar-color: #44475A;
@@ -156,7 +156,7 @@
   --tertiary-text-color: #a6adc8;
   --title-color: var(--accent-color);
   --bg-color: #1e1e2e;
-  --favorite-icon-color: #f9e2af;
+  --favorite-icon-color: #00FF00;
   --card-bg-color: #181825;
   --secondary-card-bg-color: #1e1e2e;
   --scrollbar-color: #313244;
@@ -197,7 +197,7 @@
   --tertiary-text-color: #FFFF;
   --title-color: #000000;
   --bg-color: #ff008a;
-  --favorite-icon-color: #FFEA00;
+  --favorite-icon-color: #00FF00;
   --card-bg-color: #DE1C85;
   --secondary-card-bg-color: rgba(0, 0, 0, 0.75);
   --scrollbar-color: #FFFFFF;
@@ -248,7 +248,7 @@ it can be safely elided. This looks quite pleasant on this theme. */
   --tertiary-text-color: #EEEEEE;
   --title-color: #EEEEEE;
   --bg-color: #2b2f3a;
-  --favorite-icon-color: #FFEA00;
+  --favorite-icon-color: #00FF00;
   --card-bg-color: #2e3440;
   --secondary-card-bg-color: rgba(59, 66, 82, 0.75);
   --scrollbar-color: #4b566a;