Browse Source

manually merged play button into video title

Brandon Wong 1 year ago
parent
commit
fee19d75e5
3 changed files with 229 additions and 20 deletions
  1. 218 0
      2025-04-19-13-claude-merge-play-button-rejected.md
  2. 7 10
      popup/popup.css
  3. 4 10
      popup/popup.html

File diff suppressed because it is too large
+ 218 - 0
2025-04-19-13-claude-merge-play-button-rejected.md


+ 7 - 10
popup/popup.css

@@ -61,12 +61,17 @@ h2 {
     white-space: nowrap;
 }
 
+a.video-title {
+    text-decoration: none;
+    color: #333;
+}
+
 .video-actions {
     display: flex;
     gap: 8px;
 }
 
-button, a.play-btn {
+button {
     padding: 4px 8px;
     border: none;
     border-radius: 4px;
@@ -74,20 +79,12 @@ button, a.play-btn {
     font-size: 12px;
 }
 
-.play-btn {
-    background-color: #4285f4;
-    color: white;
-    text-decoration: none;
-    display: inline-block;
-    text-align: center;
-}
-
 .remove-btn {
     background-color: #f44336;
     color: white;
 }
 
-button:hover, a.play-btn:hover {
+button:hover {
     opacity: 0.9;
 }
 

+ 4 - 10
popup/popup.html

@@ -30,19 +30,13 @@
                   :data-playlist-index="index"
                   x-bind="videoItemClass"
                 >
-                  <span
+                  <a
                     class="video-title"
+                    :href="video.url"
                     x-text="video.title"
-                    x-bind="videotitle"
-                  ></span>
+                    x-bind="videoPlayLink"
+                  ></a>
                   <div class="video-actions">
-                    <a
-                      :href="video.url"
-                      class="play-btn"
-                      x-bind="videoPlayLink"
-                    >
-                      ⏵
-                    </a>
                     <button
                       :data-playlist-name="playlistName"
                       :data-playlist-index="index"