Fix "Play on website" button style (#636)

adrienpoly · Feb 5, 2025 · 271b8b8 · 271b8b8
1 parent 660bc1f
commit 271b8b8
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 5 deletions.
diff --git a/app/assets/images/icons/fontawesome/website-brands-solid.svg b/app/assets/images/icons/fontawesome/website-brands-solid.svg
diff --git a/app/views/talks/_talk.html.erb b/app/views/talks/_talk.html.erb
@@ -104,11 +104,12 @@
           <% if talk.static_metadata&.external_player_url.present? || talk.parent_talk&.static_metadata&.external_player_url.present? %>
             <% player_url = talk.parent_talk&.static_metadata&.external_player_url || talk.static_metadata.external_player_url %>
 
-            <a class="tooltip tooltip-top" data-tip="Play on external player website" href="<%= player_url %>" target="_blank" data-action="click->video-player#pause">
-              <div class="flex gap-2 items-center rounded-full border bg-gray-100 hover:bg-gray-200/50 border-gray-200 font-regular px-3 py-1 text-sm cursor-pointer">
-                <span class="text-xs text-nowrap">Play on Website</span>
-              </div>
-            </a>
+            <%= ui_tooltip "Watch on Website" do %>
+              <%= ui_button url: player_url, kind: :pill, target: "_blank", data: {action: "click->video-player#pause"} do %>
+                <%= fa("website-brands", size: :xs, style: :solid) %>
+                <span class="text-xs">Play on Website</span>
+              <% end %>
+            <% end %>
           <% end %>
 
           <% if signed_in? && !talk.scheduled? %>