Fix context menus and menu icon visibility (#591)

* fix context menu visibility on user profiles * keep context menus larger on mobile * hide menu icon before launch
BanchanArt · Aug 10, 2023 · a4b4903 · a4b4903
1 parent 11090aa
commit a4b4903
Show file tree

Hide file tree

Showing 5 changed files with 10 additions and 6 deletions.
diff --git a/lib/banchan_web/components/nav.ex b/lib/banchan_web/components/nav.ex
@@ -19,7 +19,11 @@ defmodule BanchanWeb.Components.Nav do
       role="navigation"
       aria-label="main navigation"
     >
-      <div class="items-center flex-none gap-4 lg:hidden">
+      {!-- # TODO: remove this basic auth check on launch --}
+      <div
+        :if={@current_user || is_nil(Application.get_env(:banchan, :basic_auth))}
+        class="items-center flex-none gap-4 lg:hidden"
+      >
         <label for="drawer-toggle" class="btn btn-square btn-ghost">
           <Icon name="menu" size="6" />
         </label>

diff --git a/lib/banchan_web/live/commission_live/components/comment.ex b/lib/banchan_web/live/commission_live/components/comment.ex
@@ -173,7 +173,7 @@ defmodule BanchanWeb.CommissionLive.Components.Comment do
               </label>
               <ol
                 tabindex="0"
-                class="p-1 border dropdown-content menu bg-base-300 border-base-content border-opacity-10 rounded-xl"
+                class="p-1 border dropdown-content menu md:menu-compact bg-base-300 border-base-content border-opacity-10 rounded-xl"
               >
                 {#if Ecto.assoc_loaded?(@event.history)}
                   {#for history <- @event.history}
@@ -204,7 +204,7 @@ defmodule BanchanWeb.CommissionLive.Components.Comment do
           </label>
           <ul
             tabindex="0"
-            class="p-1 border dropdown-content menu bg-base-300 border-base-content border-opacity-10 rounded-xl"
+            class="p-1 border dropdown-content menu md:menu-compact bg-base-300 border-base-content border-opacity-10 rounded-xl"
           >
             {#if !@changeset &&
                 (@current_user_member? || @current_user.id == @actor.id || :admin in @current_user.roles ||

diff --git a/lib/banchan_web/live/denizen_live/show.ex b/lib/banchan_web/live/denizen_live/show.ex
@@ -193,7 +193,7 @@ defmodule BanchanWeb.DenizenLive.Show do
                   </label>
                   <ul
                     tabindex="0"
-                    class="p-2 border bg-base-300 border-base-content border-opacity-10 menu rounded-box"
+                    class="p-2 border menu md:menu-compact dropdown-content bg-base-300 border-base-content border-opacity-10 rounded-box"
                   >
                     {#if :admin in @current_user.roles || :mod in @current_user.roles}
                       <li>

diff --git a/lib/banchan_web/live/offering_live/show.ex b/lib/banchan_web/live/offering_live/show.ex
@@ -257,7 +257,7 @@ defmodule BanchanWeb.OfferingLive.Show do
                   </label>
                   <ul
                     tabindex="0"
-                    class="p-1 border dropdown-content menu bg-base-100 border-base-content border-opacity-10 rounded-xl"
+                    class="p-1 border dropdown-content menu md:menu-compact bg-base-100 border-base-content border-opacity-10 rounded-xl"
                   >
                     {#if @current_user && (@current_user_member? || Accounts.mod?(@current_user))}
                       <li>

diff --git a/lib/banchan_web/live/studio_live/components/studio_layout.ex b/lib/banchan_web/live/studio_live/components/studio_layout.ex
@@ -105,7 +105,7 @@ defmodule BanchanWeb.StudioLive.Components.StudioLayout do
                   </label>
                   <ul
                     tabindex="0"
-                    class="p-1 border rounded-xl dropdown-content bg-base-300 border-base-content border-opacity-10 menu menu-compact"
+                    class="p-1 border rounded-xl dropdown-content bg-base-300 border-base-content border-opacity-10 menu md:menu-compact"
                   >
                     {#if Accounts.mod?(@current_user)}
                       <li>