From a9c350c6224f08818cd50fdab727409f6e0017ba Mon Sep 17 00:00:00 2001
From: Miguel Michelson <miguelmichelson@gmail.com>
Date: Sun, 14 Jul 2024 00:24:03 -0400
Subject: [PATCH] form submit, errors

---
 app/views/products/_form.erb            | 36 ++++++++++++++++---------
 app/views/products/_product_images.erb  |  2 +-
 app/views/products/_shipping_fields.erb | 12 ++++++---
 3 files changed, 34 insertions(+), 16 deletions(-)

diff --git a/app/views/products/_form.erb b/app/views/products/_form.erb
index 828a39b..3d5903d 100644
--- a/app/views/products/_form.erb
+++ b/app/views/products/_form.erb
@@ -108,7 +108,7 @@
 
       <% # render "variant_form_section", form: form %>
 
-      <div class="border p-4 my-4">
+      <div class="border border-muted rounded-lg p-4 my-4">
         <h3 class="text-lg">Shipping</h3>
       
         <div>
@@ -117,6 +117,15 @@
           <span>days of being placed</span>
         </div>
 
+        <div>
+          <%= form.number_field :shipping_days, label: "Orders ship out within", class: "mt-1 block w-full rounded-md border-gray-300 shadow-sm focus:border-indigo-300 focus:ring focus:ring-indigo-200 focus:ring-opacity-50" %>
+          <span class="mt-1 text-sm text-gray-500">days of being placed</span>
+        </div>
+
+        <div>
+          <%= form.date_field :shipping_begins_on, label: "Begins shipping on", class: "mt-1 block w-full rounded-md border-gray-300 shadow-sm focus:border-indigo-300 focus:ring focus:ring-indigo-200 focus:ring-opacity-50" %>
+        </div>
+
         <div data-controller="nested-form" class="py-6 sm:col-span-6">
           <template data-nested-form-target="template">
             <%= form.fields_for :product_shippings, ProductShipping.new, child_index: 'NEW_RECORD' do |ff| %>
@@ -124,15 +133,6 @@
             <% end %>
           </template>
 
-          <div>
-            <%= form.number_field :shipping_days, label: "Orders ship out within", class: "mt-1 block w-full rounded-md border-gray-300 shadow-sm focus:border-indigo-300 focus:ring focus:ring-indigo-200 focus:ring-opacity-50" %>
-            <span class="mt-1 text-sm text-gray-500">days of being placed</span>
-          </div>
-
-          <div>
-            <%= form.date_field :shipping_begins_on, label: "Begins shipping on", class: "mt-1 block w-full rounded-md border-gray-300 shadow-sm focus:border-indigo-300 focus:ring focus:ring-indigo-200 focus:ring-opacity-50" %>
-          </div>
-
           <!--<div>
             <h3 class="text-lg font-medium text-gray-900">Shipping Prices</h3>
             <div>
@@ -158,8 +158,20 @@
       </div>
 
  
-      <div class="py-2">
-        <%= form.submit %>
+      <div class="py-2 fixed bottom-[65px] w-full bg-muted left-0 border-t border-muted">
+       
+        <div class="flex space-x-2 items-center p-6">
+          
+          <%= form.submit %>
+
+          <% if form.object.errors.any? %>
+            <div class="py-3">
+              <p class="text-danger text-red-500 text-2xl">Errors Found</p>
+              <%= form.object.errors.full_messages.join(" ") %>
+            </div>
+          <% end %>
+
+        </div>
       </div>
     </div>
 
diff --git a/app/views/products/_product_images.erb b/app/views/products/_product_images.erb
index bf43ca6..4ab0127 100644
--- a/app/views/products/_product_images.erb
+++ b/app/views/products/_product_images.erb
@@ -1,4 +1,4 @@
-<div class="nested-fields border mt-6 p-4"
+<div class="nested-fields border mt-6 p-4 border-muted rounded-lg"
 	data-new-record="<%= form.object.new_record? %>">
 
   <div class="flex space-x-2">
diff --git a/app/views/products/_shipping_fields.erb b/app/views/products/_shipping_fields.erb
index e296c0c..bc123ff 100644
--- a/app/views/products/_shipping_fields.erb
+++ b/app/views/products/_shipping_fields.erb
@@ -1,4 +1,4 @@
-<div class="nested-fields border mt-6 p-4"
+<div class="nested-fields mt-6 p-4 bg-muted rounded-lg"
 	data-new-record="<%= form.object.new_record? %>">
 
   <div class="flex items-center space-x-2">
@@ -7,11 +7,17 @@
       <%= form.country_select :country, 
         disabled: form.object.is_default?, 
         priority_countries: ["CL", "AR", "PE", "BO", "CO", "MX", "US", "BR", "ES", "FR", "GR"],
-        except: %w[AS CX CC CU HM IR KP MH FM NF MP PW SD SY UM VI] 
+        except: %w[AS CX CC CU HM IR KP MH FM NF MP PW SD SY UM VI]
       %>
+
+      <div class="mt-1 text-xs text-gray-500 dark:text-gray-100">
+        <div class="text-gray-500 text-xs">
+          Shipping country
+        </div>
+      </div>
     </div>
     
-    <%= form.number_field :base_cost, step: 0.01, label: "USD for 1 unit"  %>
+    <%= form.number_field :base_cost, step: 0.01, label: "USD for 1 unit", hint: "Base cost"  %>
     
     <%= form.number_field :additional_cost, step: 0.01, hint:  "USD more per additional unit" %>
   </div>