From 2c6abb32cae188bf6d47bc3f77f3cee88d309e86 Mon Sep 17 00:00:00 2001
From: Miguel Ángel Moreno <mail@migalmoreno.com>
Date: Sun, 1 Dec 2024 22:42:15 +0100
Subject: feat: remove buttons padding and use flex gap

---
 src/frontend/tubo/bg_player/views.cljs | 11 +++++------
 src/frontend/tubo/queue/views.cljs     |  2 +-
 src/frontend/tubo/stream/views.cljs    |  2 +-
 3 files changed, 7 insertions(+), 8 deletions(-)

diff --git a/src/frontend/tubo/bg_player/views.cljs b/src/frontend/tubo/bg_player/views.cljs
index 6be514e..c510731 100644
--- a/src/frontend/tubo/bg_player/views.cljs
+++ b/src/frontend/tubo/bg_player/views.cljs
@@ -59,7 +59,7 @@
 
 (defn button
   [& {:keys [icon on-click disabled? show-on-mobile? extra-classes]}]
-  [:button.outline-none.focus:ring-transparent.px-2.pt-1
+  [:button.outline-none.focus:ring-transparent
    {:class    (into (into (when disabled? [:opacity-50 :cursor-auto])
                           (when-not show-on-mobile? [:hidden :lg:block]))
                     extra-classes)
@@ -124,8 +124,7 @@
          [button
           :icon
           (if muted? [:i.fa-solid.fa-volume-xmark] [:i.fa-solid.fa-volume-low])
-          :on-click #(rf/dispatch [:bg-player/mute (not muted?) player])
-          :extra-classes [:pl-3 :pr-2]]
+          :on-click #(rf/dispatch [:bg-player/mute (not muted?) player])]
          (when @show-slider?
            [:input.absolute.w-24.ml-2.m-1.bottom-16
             {:class    (str/join " " styles)
@@ -162,7 +161,7 @@
         paused?          @(rf/subscribe [:player/paused])
         !elapsed-time    @(rf/subscribe [:elapsed-time])]
     [:div.flex.flex-col.items-center.ml-auto
-     [:div.flex.justify-end
+     [:div.flex.justify-end.gap-x-4
       [loop-button loop-playback color]
       [button
        :icon [:i.fa-solid.fa-backward-step]
@@ -215,7 +214,7 @@
                                 first
                                 :items))
             bookmark  #(rf/dispatch [:modals/open [modals/add-to-bookmark %]])]
-        [:div.flex.lg:justify-end.lg:flex-1
+        [:div.flex.lg:justify-end.lg:flex-1.gap-x-2
          [volume-slider !player volume muted? color]
          [button
           :icon [:i.fa-solid.fa-list]
@@ -253,7 +252,7 @@
             :icon     [:i.fa-solid.fa-close]
             :on-click #(rf/dispatch [:bg-player/dispose])}]
           :menu-styles {:bottom "30px" :top nil :right "10px"}
-          :extra-classes [:pt-1 :!pl-4 :px-3]]]))))
+          :extra-classes [:!pl-4 :px-3]]]))))
 
 (defn audio-player
   [_]
diff --git a/src/frontend/tubo/queue/views.cljs b/src/frontend/tubo/queue/views.cljs
index df924a5..7f7a874 100644
--- a/src/frontend/tubo/queue/views.cljs
+++ b/src/frontend/tubo/queue/views.cljs
@@ -102,7 +102,7 @@
        (if (and bg-player-ready? @!player)
          (utils/format-duration (.-duration @!player))
          "--:--")]]
-     [:div.flex.justify-center.items-center
+     [:div.flex.justify-center.items-center.gap-x-4
       [player/loop-button loop-playback color true]
       [player/button
        :icon [:i.fa-solid.fa-backward-step]
diff --git a/src/frontend/tubo/stream/views.cljs b/src/frontend/tubo/stream/views.cljs
index 51896f0..7e03f40 100644
--- a/src/frontend/tubo/stream/views.cljs
+++ b/src/frontend/tubo/stream/views.cljs
@@ -124,7 +124,7 @@
            [:div.flex.flex-wrap.items-center.justify-between.mt-8.min-w-full
             [:div.flex.gap-2.items-center
              [:span.font-semibold.text-xl "Next Up"]]
-            [:div.flex.gap-4
+            [:div.flex.gap-x-4
              [layout/popover-menu !menu-active?
               [{:label    "Add to queue"
                 :icon     [:i.fa-solid.fa-headphones]
-- 
cgit v1.2.3