From a242e657336dffa692a5c96fb5df1428d372feaf Mon Sep 17 00:00:00 2001
From: Barry Hoggard <bhoggard@gmail.com>
Date: Mon, 23 Dec 2024 20:24:43 -0500
Subject: [PATCH 1/4] allow profile edit for admins too

---
 app/models/user.rb | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/app/models/user.rb b/app/models/user.rb
index fff3c20..da8ea5b 100644
--- a/app/models/user.rb
+++ b/app/models/user.rb
@@ -34,7 +34,9 @@ def show_users?
   end
 
   def has_profile?
-    artist?
+    return false unless profile
+
+    artist? || admin?
   end
 
   def active_for_authentication?

From 36eb8a03993658f4b1083f700a97c7d2cccb9da1 Mon Sep 17 00:00:00 2001
From: sepans <sepans@sepans.com>
Date: Wed, 25 Dec 2024 09:48:04 -0500
Subject: [PATCH 2/4] Add price comma, remove price from main page

---
 app/views/artworks/_info.html.erb | 2 +-
 app/views/main/_artwork.html.erb  | 1 -
 app/views/main/_info.html.erb     | 2 +-
 3 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/app/views/artworks/_info.html.erb b/app/views/artworks/_info.html.erb
index 855587b..8a0d814 100644
--- a/app/views/artworks/_info.html.erb
+++ b/app/views/artworks/_info.html.erb
@@ -19,7 +19,7 @@
   </div>
   <% if @artwork.price.present? %>
     <div class="text-xl py-2">
-      Price: <%= @artwork.price %>
+      Price: <%= number_with_delimiter(@artwork.price) %>
     </div>
   <% end %>
   <% if @artwork.description.present? %>
diff --git a/app/views/main/_artwork.html.erb b/app/views/main/_artwork.html.erb
index 99ee704..b5c32bc 100644
--- a/app/views/main/_artwork.html.erb
+++ b/app/views/main/_artwork.html.erb
@@ -6,7 +6,6 @@
     <section class="p-4">
       <div class=""><%= artwork.artist_name %></div>
       <div class="text-gray-500 italic"><%= artwork.title %></div>
-      <div class="text-black text-lg mb-1">Price: <%= artwork.price.present? ? artwork.price : "N/A" %></div>
       <div class=" text-sm text-gray-700"><%= simple_format artwork.description %></div>
     </section>
   </a>
diff --git a/app/views/main/_info.html.erb b/app/views/main/_info.html.erb
index 6615853..d2c430f 100644
--- a/app/views/main/_info.html.erb
+++ b/app/views/main/_info.html.erb
@@ -18,7 +18,7 @@
     Location: <%= @artwork.location %>
   </div>
   <div class="text-2xl py-2">
-    Price: <%= @artwork.price.present? ? @artwork.price : "N/A" %>
+    Price: <%= @artwork.price.present? ? number_with_delimiter(@artwork.price) : "N/A" %>
   </div>
   <% if @artwork.description.present? %>
     <div class="mt-2 text-slate-500">

From e16c7132318620ba24aa80abfe367d2c4199b900 Mon Sep 17 00:00:00 2001
From: sepans <sepans@sepans.com>
Date: Wed, 25 Dec 2024 10:02:35 -0500
Subject: [PATCH 3/4] just break lines

---
 app/views/main/show.html.erb | 12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)

diff --git a/app/views/main/show.html.erb b/app/views/main/show.html.erb
index c346d17..0f211f3 100644
--- a/app/views/main/show.html.erb
+++ b/app/views/main/show.html.erb
@@ -45,7 +45,11 @@
 
       <!--Carousel controls - prev item-->
       <button
-        class="absolute bottom-0 left-0 top-0 z-[1] flex w-[15%] items-center justify-center border-0 bg-none p-0 text-center text-white opacity-50 transition-opacity duration-150 ease-[cubic-bezier(0.25,0.1,0.25,1.0)] hover:text-white hover:no-underline hover:opacity-90 hover:outline-none focus:text-white focus:no-underline focus:opacity-90 focus:outline-none motion-reduce:transition-none"
+        class="absolute bottom-0 left-0 top-0 z-[1] flex w-[15%] items-center justify-center
+          border-0 bg-none p-0 text-center text-white opacity-50 transition-opacity duration-150 
+          ease-[cubic-bezier(0.25,0.1,0.25,1.0)] hover:text-white hover:no-underline hover:opacity-90
+          hover:outline-none focus:text-white focus:no-underline focus:opacity-90 focus:outline-none
+          motion-reduce:transition-none"
         type="button"
         data-carousel-prev>
         <span class="inline-block h-8 w-8">
@@ -69,7 +73,11 @@
       </button>
       <!--Carousel controls - next item-->
       <button
-        class="absolute bottom-0 right-0 top-0 z-[1] flex w-[15%] items-center justify-center border-0 bg-none p-0 text-center text-white opacity-50 transition-opacity duration-150 ease-[cubic-bezier(0.25,0.1,0.25,1.0)] hover:text-white hover:no-underline hover:opacity-90 hover:outline-none focus:text-white focus:no-underline focus:opacity-90 focus:outline-none motion-reduce:transition-none"
+        class="absolute bottom-0 right-0 top-0 z-[1] flex w-[15%] items-center justify-center border-0
+          bg-none p-0 text-center text-white opacity-50 transition-opacity duration-150 
+          ease-[cubic-bezier(0.25,0.1,0.25,1.0)] hover:text-white hover:no-underline hover:opacity-90
+          hover:outline-none focus:text-white focus:no-underline focus:opacity-90 focus:outline-none
+          motion-reduce:transition-none"
         type="button"
         data-carousel-next>
         <span class="inline-block h-8 w-8">

From 72da781e20cbceff5bd489e5eb7c3e1e735fe160 Mon Sep 17 00:00:00 2001
From: sepans <sepans@sepans.com>
Date: Thu, 26 Dec 2024 03:50:35 -0500
Subject: [PATCH 4/4] move caption and lil rects below artwork image

---
 app/views/main/show.html.erb | 29 ++++++++++++++++-------------
 1 file changed, 16 insertions(+), 13 deletions(-)

diff --git a/app/views/main/show.html.erb b/app/views/main/show.html.erb
index 0f211f3..055725f 100644
--- a/app/views/main/show.html.erb
+++ b/app/views/main/show.html.erb
@@ -10,17 +10,18 @@
 
       <!--Carousel items-->
       <div
-        class="relative w-full overflow-hidden after:clear-both after:block after:content-['']">
+        class="relative w-full after:clear-both after:block after:content-['']">
         <% @artwork.images.each_with_index do | image, i | %>
         <a data-fslightbox href="<%= image.file.url %>">
           <div
-            class="relative float-left -mr-[100%] w-full transition-opacity duration-[600ms] ease-in-out motion-reduce:transition-none opacity-0 pointer-events-none <%= "!opacity-100 pointer-events-auto" if i == 0 %>"
+            class="relative float-left -mr-[100%] w-full transition-opacity duration-[300ms] ease-in-out motion-reduce:transition-none
+                   opacity-0 cursor-default <%= "!opacity-100 pointer-events-auto" if i == 0 %>"
             data-carousel-item
             style="backface-visibility: hidden">
             <%= image_tag image.file.url, alt: image.caption, class: "block w-full" %>
             <div
-              class="absolute inset-x-[15%] bottom-0 py-5 text-center text-white block">
-              <h5 class="text-slate-200"><%= image.caption %></h5>
+              class="absolute inset-x-[15%] -bottom-16 <%= "-bottom-20" if @artwork.images.length > 1 %> py-5 text-center block">
+              <h5 class="text-black"><%= image.caption %></h5>
 
             </div>
           </div>    
@@ -30,14 +31,14 @@
       <% if @artwork.images.length > 1 %>
       <!--Carousel indicators-->
       <div
-        class="absolute bottom-6 left-0 right-0 z-[2] mx-[15%] mb-4 flex list-none justify-center p-0"
+        class="absolute -bottom-12 left-0 right-0 z-[2] mx-[15%] mb-4 flex list-none justify-center p-0"
         data-twe-carousel-indicators>
         <% @artwork.images.each_with_index do |artwork, i| %>
           <button
             type="button"
             aria-label="Slide <%= i %>" data-carousel-slide-to="<%= i %>"
             class="mx-[3px] box-content h-[3px] w-[30px] flex-initial cursor-pointer 
-                  border-0 border-y-[10px] border-solid border-transparent bg-white
+                  border-0 border-y-[10px] border-solid border-transparent bg-black
                   bg-clip-padding p-0 -indent-[999px] opacity-50 transition-opacity
                   duration-[600ms] ease-[cubic-bezier(0.25,0.1,0.25,1.0)] motion-reduce:transition-none <%= '!opacity-100' if i == 0 %>"></button>
         <% end %>
@@ -105,15 +106,17 @@
         
   </div>
 
-  <div class="lg:row-start-1 lg:col-start-4 lg:row-span-2">
+  <div class="lg:row-start-1 lg:col-start-4 lg:row-span-2 mt-8 <%= "mt-12" if @artwork.images.length > 1 %>">
     <%= render "info" %>
   </div>
-  <section class="lg:col-span-3 lg:row-start-3 lg:row-span-4">
-      <div class="pb-2">
-      About artist:
-      </div>
-      <%= simple_format(@artwork.user.profile&.bio) %>
-  </section>  
+  <% if @artwork.user.profile&.bio&.present? %>
+    <section class="mt-10 lg:col-span-3 lg:row-start-3 lg:row-span-4">
+        <div class="pb-2">
+        About artist:
+        </div>
+        <%= simple_format(@artwork.user.profile&.bio) %>
+    </section>
+  <% end %>
 </div>
 
 <%= javascript_include_tag "lightbox-refresh" %>