From 36fe8937a7642e4f5ad3a0bccb3135a39442e77a Mon Sep 17 00:00:00 2001
From: okbel <curciobel@gmail.com>
Date: Tue, 1 Jun 2021 09:34:54 -0300
Subject: [PATCH] sidebar cart

---
 components/cart/CartItem/CartItem.module.css  |  5 ++++
 components/cart/CartItem/CartItem.tsx         | 24 +++++++++----------
 .../CheckoutSidebarView.tsx                   | 12 ++++++----
 3 files changed, 24 insertions(+), 17 deletions(-)

diff --git a/components/cart/CartItem/CartItem.module.css b/components/cart/CartItem/CartItem.module.css
index a4d62aeca..feffa0f3a 100644
--- a/components/cart/CartItem/CartItem.module.css
+++ b/components/cart/CartItem/CartItem.module.css
@@ -41,3 +41,8 @@
   left: 30% !important;
   top: 30% !important;
 }
+
+.productName {
+  @apply font-medium cursor-pointer pb-1;
+  margin-top: -4px;
+}
diff --git a/components/cart/CartItem/CartItem.tsx b/components/cart/CartItem/CartItem.tsx
index 990c94a17..c03c2a179 100644
--- a/components/cart/CartItem/CartItem.tsx
+++ b/components/cart/CartItem/CartItem.tsx
@@ -19,11 +19,11 @@ type ItemOption = {
 
 const CartItem = ({
   item,
+  variant = 'default',
   currencyCode,
-  noEdit = false,
   ...rest
 }: {
-  noEdit?: boolean
+  variant: 'default' | 'display'
   item: LineItem
   currencyCode: string
 }) => {
@@ -46,7 +46,6 @@ const CartItem = ({
 
   const handleQuantity = (e: ChangeEvent<HTMLInputElement>) => {
     const val = Number(e.target.value)
-
     if (Number.isInteger(val) && val >= 0) {
       setQuantity(Number(e.target.value))
     }
@@ -54,7 +53,6 @@ const CartItem = ({
 
   const handleBlur = () => {
     const val = Number(quantity)
-
     if (val !== item.quantity) {
       updateQuantity(val)
     }
@@ -62,7 +60,6 @@ const CartItem = ({
 
   const increaseQuantity = (n = 1) => {
     const val = Number(quantity) + n
-
     if (Number.isInteger(val) && val >= 0) {
       setQuantity(val)
       updateQuantity(val)
@@ -71,7 +68,6 @@ const CartItem = ({
 
   const handleRemove = async () => {
     setRemoving(true)
-
     try {
       // If this action succeeds then there's no need to do `setRemoving(true)`
       // because the component will be removed from the view
@@ -80,6 +76,7 @@ const CartItem = ({
       setRemoving(false)
     }
   }
+
   // TODO: Add a type for this
   const options = (item as any).options
 
@@ -114,14 +111,14 @@ const CartItem = ({
         <div className="flex-1 flex flex-col text-base">
           <Link href={`/product/${item.path}`}>
             <span
-              className="font-medium cursor-pointer leading-6"
+              className={s.productName}
               onClick={() => closeSidebarIfPresent()}
             >
               {item.name}
             </span>
           </Link>
-          {options && options.length > 0 ? (
-            <div className="">
+          {options && options.length > 0 && (
+            <div className="flex items-center pb-1">
               {options.map((option: ItemOption, i: number) => (
                 <div
                   key={`${item.id}-${option.name}`}
@@ -144,13 +141,16 @@ const CartItem = ({
                 </div>
               ))}
             </div>
-          ) : null}
+          )}
+          {variant === 'display' && (
+            <div className="text-sm tracking-wider">{quantity}x</div>
+          )}
         </div>
         <div className="flex flex-col justify-between space-y-2 text-sm">
           <span>{price}</span>
         </div>
       </div>
-      {!noEdit ? (
+      {variant === 'default' && (
         <div className="flex flex-row h-9">
           <button className={s.actions} onClick={handleRemove}>
             <Cross width={20} height={20} />
@@ -183,8 +183,6 @@ const CartItem = ({
             <Plus width={18} height={18} />
           </button>
         </div>
-      ) : (
-        <div>x{quantity}</div>
       )}
     </li>
   )
diff --git a/components/checkout/CheckoutSidebarView/CheckoutSidebarView.tsx b/components/checkout/CheckoutSidebarView/CheckoutSidebarView.tsx
index 9d7363290..c9dab3d83 100644
--- a/components/checkout/CheckoutSidebarView/CheckoutSidebarView.tsx
+++ b/components/checkout/CheckoutSidebarView/CheckoutSidebarView.tsx
@@ -144,13 +144,13 @@ const CheckoutSidebarView: FC = () => {
               </div>
             </div>
 
-            <ul className="py-4 space-y-6 sm:py-0 sm:space-y-0 sm:divide-y sm:divide-accent-3 border-accent-3">
+            <ul className="py-4 space-y-6 sm:py-0 sm:space-y-0 sm:divide-y sm:divide-accent-2 border-accent-2">
               {data!.lineItems.map((item: any) => (
                 <CartItem
                   key={item.id}
                   item={item}
                   currencyCode={data!.currency.code}
-                  noEdit
+                  variant="display"
                 />
               ))}
             </ul>
@@ -171,13 +171,17 @@ const CheckoutSidebarView: FC = () => {
                 <span className="font-bold tracking-wide">FREE</span>
               </li>
             </ul>
-            <div className="flex justify-between border-t border-accent-3 py-3 font-bold mb-2">
+            <div className="flex justify-between border-t border-accent-2 py-3 font-bold mb-2">
               <span>Total</span>
               <span>{total}</span>
             </div>
             <div>
-              <Button Component="a" width="100%">
+              {/* Once data is correcly filled */}
+              {/* <Button Component="a" width="100%">
                 Confirm Purchase
+              </Button> */}
+              <Button Component="a" width="100%" variant="ghost">
+                Continue
               </Button>
             </div>
           </div>