From 202205c03857d26ae61f3d7fa3bfea58925f6520 Mon Sep 17 00:00:00 2001
From: ncoronel <ncoronel@quo.ar>
Date: Thu, 2 Jul 2026 09:01:30 -0300
Subject: [PATCH] feat: integrate quantity selector component into cart and
 product detail pages for enhanced functionality

---
 .../product-detail-page.component.html        |  5 +---
 .../product-detail-page.component.ts          |  7 ------
 .../cart-item/cart-item.component.html        | 14 +++++++----
 .../cart-item/cart-item.component.scss        | 13 +---------
 .../cart-item/cart-item.component.ts          | 25 ++++++++++++++++++-
 .../quantity-selector.component.html          |  2 +-
 .../quantity-selector.component.scss          | 25 +++++++++++++++++++
 .../quantity-selector.component.ts            | 12 ++++-----
 8 files changed, 66 insertions(+), 37 deletions(-)

diff --git a/src/app/features/store/pages/product-detail-page/product-detail-page.component.html b/src/app/features/store/pages/product-detail-page/product-detail-page.component.html
index f934867..cadc2f2 100644
--- a/src/app/features/store/pages/product-detail-page/product-detail-page.component.html
+++ b/src/app/features/store/pages/product-detail-page/product-detail-page.component.html
@@ -54,11 +54,8 @@
             <section class="product-detail__section">
               <div class="product-detail__purchase">
                 <app-quantity-selector
-                  [quantity]="quantity()"
+                  [(quantity)]="quantity"
                   [max]="selectedVariant()?.stock ?? 1"
-                  (increase)="increaseQuantity()"
-                  (decrease)="decreaseQuantity()"
-                  (quantityChange)="quantity.set($event)"
                 />
 
                 <div class="product-detail__actions">
diff --git a/src/app/features/store/pages/product-detail-page/product-detail-page.component.ts b/src/app/features/store/pages/product-detail-page/product-detail-page.component.ts
index 43c22eb..08eec66 100644
--- a/src/app/features/store/pages/product-detail-page/product-detail-page.component.ts
+++ b/src/app/features/store/pages/product-detail-page/product-detail-page.component.ts
@@ -215,14 +215,7 @@ export class ProductDetailPageComponent implements OnInit, OnDestroy {
     }
   }
 
-  protected increaseQuantity(): void {
-    const currentStock = this.selectedVariant()?.stock ?? 1;
-    this.quantity.update((current) => current < currentStock ? current + 1 : current);
-  }
 
-  protected decreaseQuantity(): void {
-    this.quantity.update((current) => Math.max(1, current - 1));
-  }
 
   protected addToCart(): void {
     const variant = this.selectedVariant();
diff --git a/src/app/shared/components/cart-item/cart-item.component.html b/src/app/shared/components/cart-item/cart-item.component.html
index d4c8722..0bb1480 100644
--- a/src/app/shared/components/cart-item/cart-item.component.html
+++ b/src/app/shared/components/cart-item/cart-item.component.html
@@ -33,11 +33,15 @@
       }
     </div>
 
-    <div class="d-flex justify-content-end align-items-center mt-auto cart-item-actions" aria-hidden="true">
-      <button class="border d-inline-flex align-items-center justify-content-center bg-white rounded-0 cart-item-qty-btn" type="button">-</button>
-      <span class="border d-inline-flex align-items-center justify-content-center bg-white rounded-0 cart-item-qty-value" >{{ quantity() }}</span>
-      <button class="border d-inline-flex align-items-center justify-content-center bg-white rounded-0 cart-item-qty-btn" type="button">+</button>
-      <button class="border-0 bg-transparent d-inline-flex align-items-center justify-content-center cart-item-remove-btn" type="button">
+    <div class="d-flex justify-content-end align-items-center mt-auto cart-item-actions">
+      <app-quantity-selector
+        size="small"
+        [quantity]="quantity()"
+        (quantityChange)="onQuantityChange($event)"
+        (increase)="onIncrease()"
+        (decrease)="onDecrease()"
+      />
+      <button class="border-0 bg-transparent d-inline-flex align-items-center justify-content-center cart-item-remove-btn" type="button" (click)="onRemove()">
         <i class="fa-solid fa-trash"></i>
       </button>
     </div>
diff --git a/src/app/shared/components/cart-item/cart-item.component.scss b/src/app/shared/components/cart-item/cart-item.component.scss
index a13197a..7553a99 100644
--- a/src/app/shared/components/cart-item/cart-item.component.scss
+++ b/src/app/shared/components/cart-item/cart-item.component.scss
@@ -92,19 +92,8 @@
   gap: 0.125rem;
 }
 
-.cart-item-qty-btn,
-.cart-item-qty-value {
-  width: 18px;
-  height: 18px;
-  border-color: #cfcfcf !important;
-  color: #666666;
-  font-size: 10px;
-  line-height: 1;
-}
 
-.cart-item-qty-btn {
-  padding: 0;
-}
+
 
 .cart-item-remove-btn {
   width: 18px;
diff --git a/src/app/shared/components/cart-item/cart-item.component.ts b/src/app/shared/components/cart-item/cart-item.component.ts
index aa1134e..68465cb 100644
--- a/src/app/shared/components/cart-item/cart-item.component.ts
+++ b/src/app/shared/components/cart-item/cart-item.component.ts
@@ -1,4 +1,5 @@
-import { ChangeDetectionStrategy, Component, computed, input } from '@angular/core';
+import { ChangeDetectionStrategy, Component, computed, input, output } from '@angular/core';
+import { QuantitySelectorComponent } from '../quantity-selector/quantity-selector.component';
 
 export interface CartItemAttribute {
   label: string;
@@ -8,6 +9,7 @@ export interface CartItemAttribute {
 @Component({
   selector: 'app-cart-item',
   standalone: true,
+  imports: [QuantitySelectorComponent],
   templateUrl: './cart-item.component.html',
   styleUrl: './cart-item.component.scss',
   changeDetection: ChangeDetectionStrategy.OnPush
@@ -21,6 +23,27 @@ export class CartItemComponent {
   readonly attributes = input<CartItemAttribute[]>([]);
   readonly quantity = input<number>(1);
 
+  readonly quantityChange = output<number>();
+  readonly remove = output<void>();
+  readonly increase = output<void>();
+  readonly decrease = output<void>();
+
+  protected onQuantityChange(newQuantity: number): void {
+    this.quantityChange.emit(newQuantity);
+  }
+
+  protected onRemove(): void {
+    this.remove.emit();
+  }
+
+  protected onIncrease(): void {
+    this.increase.emit();
+  }
+
+  protected onDecrease(): void {
+    this.decrease.emit();
+  }
+
   protected readonly formattedOriginalPrice = computed(() => {
     const price = this.originalPrice();
     return price === null ? null : this.formatCurrency(price);
diff --git a/src/app/shared/components/quantity-selector/quantity-selector.component.html b/src/app/shared/components/quantity-selector/quantity-selector.component.html
index c991d01..a9614a7 100644
--- a/src/app/shared/components/quantity-selector/quantity-selector.component.html
+++ b/src/app/shared/components/quantity-selector/quantity-selector.component.html
@@ -1,4 +1,4 @@
-<div class="quantity-selector" aria-label="Selector de cantidad">
+<div class="quantity-selector" [class.quantity-selector--small]="size() === 'small'" aria-label="Selector de cantidad">
   <button
     type="button"
     class="quantity-selector__button"
diff --git a/src/app/shared/components/quantity-selector/quantity-selector.component.scss b/src/app/shared/components/quantity-selector/quantity-selector.component.scss
index 580c737..de273f9 100644
--- a/src/app/shared/components/quantity-selector/quantity-selector.component.scss
+++ b/src/app/shared/components/quantity-selector/quantity-selector.component.scss
@@ -44,4 +44,29 @@
     color: #6f6f6f;
     user-select: none;
   }
+
+  &--small {
+    min-height: 18px;
+    border-radius: 0px;
+    border: 1px solid #cfcfcf;
+    background: #ffffff;
+
+    .quantity-selector__button {
+      width: 18px;
+      height: 18px;
+      font-size: 10px;
+      background: #ffffff;
+      color: #666666;
+
+      &:not(:disabled):hover {
+        background-color: #f0f0f0;
+      }
+    }
+
+    .quantity-selector__value {
+      min-width: 18px;
+      font-size: 10px;
+      color: #666666;
+    }
+  }
 }
diff --git a/src/app/shared/components/quantity-selector/quantity-selector.component.ts b/src/app/shared/components/quantity-selector/quantity-selector.component.ts
index 6ae9aee..ec121a1 100644
--- a/src/app/shared/components/quantity-selector/quantity-selector.component.ts
+++ b/src/app/shared/components/quantity-selector/quantity-selector.component.ts
@@ -1,4 +1,4 @@
-import { ChangeDetectionStrategy, Component, input, output } from '@angular/core';
+import { ChangeDetectionStrategy, Component, input, model, output } from '@angular/core';
 
 @Component({
   selector: 'app-quantity-selector',
@@ -9,27 +9,25 @@ import { ChangeDetectionStrategy, Component, input, output } from '@angular/core
   changeDetection: ChangeDetectionStrategy.OnPush
 })
 export class QuantitySelectorComponent {
-  readonly quantity = input<number>(1);
+  readonly quantity = model<number>(1);
   readonly min = input<number>(1);
   readonly max = input<number>(100);
+  readonly size = input<'small' | 'medium'>('medium');
 
   readonly increase = output<void>();
   readonly decrease = output<void>();
-  readonly quantityChange = output<number>();
 
   protected onDecrease(): void {
     if (this.quantity() > this.min()) {
-      const newValue = this.quantity() - 1;
+      this.quantity.set(this.quantity() - 1);
       this.decrease.emit();
-      this.quantityChange.emit(newValue);
     }
   }
 
   protected onIncrease(): void {
     if (this.quantity() < this.max()) {
-      const newValue = this.quantity() + 1;
+      this.quantity.set(this.quantity() + 1);
       this.increase.emit();
-      this.quantityChange.emit(newValue);
     }
   }
 }