Merge branch 'homologacion'

refactor(store-header): update class names for consistency and readability
Merge pull request 'homologacion' (#9 ) from homologacion into main
2026-09-24 17:00:14 -03:00 · 2026-09-24 16:59:55 -03:00 · 2026-09-24 19:54:56 +00:00 · 2026-09-17 17:52:29 +00:00
12 changed files with 138 additions and 367 deletions
--- a/src/app/core/layout/store-layout/store-header/immersive-store-header.component.html
+++ b/src/app/core/layout/store-layout/store-header/immersive-store-header.component.html
@@ -1,8 +1,8 @@
 <header class="immersive-header" [class.immersive-header--compact]="!showHero()">
-  <div class="immersive-header__backdrop">
+  <div class="ih-backdrop">
    @if (showHero() && backgroundImageUrl(); as backgroundImage) {
      <img
-        class="immersive-header__backdrop-image"
+        class="ih-backdrop-image"
        [src]="backgroundImage"
        alt=""
        loading="eager"
@@ -12,12 +12,12 @@
    }
  </div>

-  <div class="immersive-header__content">
+  <div class="ih-content">
    <nav
-      class="immersive-header__nav container-xl px-3 px-md-4"
+      class="ih-nav container-xl px-3 px-md-4"
      aria-label="Navegación de la tienda"
    >
-      <a routerLink="/" class="immersive-header__brand" aria-label="Ir al inicio">
+      <a routerLink="/" class="ih-brand" aria-label="Ir al inicio">
        @if (logoUrl()) {
          <img
            [src]="logoUrl()"
@@ -29,12 +29,12 @@
        }
      </a>

-      <div class="immersive-header__actions">
+      <div class="ih-actions">
        @if (displaySearchBar()) {
          <form
            id="immersive-search"
-            class="immersive-header__search"
-            [class.immersive-header__search--open]="isSearchOpen()"
+            class="ih-search"
+            [class.ih-search--open]="isSearchOpen()"
            role="search"
            (submit)="submitSearch($event)"
          >
@@ -48,7 +48,7 @@
              (valueChange)="showSearchError.set(false)"
            />
            @if (showSearchError()) {
-              <span id="immersive-search-error" class="immersive-header__search-error" role="alert"
+              <span id="immersive-search-error" class="ih-search-error" role="alert"
                >Ingresá al menos 3 caracteres.</span
              >
            }
@@ -59,7 +59,7 @@
          @if (ticketsMenu(); as menu) {
            <app-button
              variant="primary"
-              hostClass="immersive-header__tickets"
+              hostClass="ih-tickets"
              data-testid="store-header-tickets"
              (click)="ticketsClick.emit()"
            >
@@ -72,7 +72,7 @@
          <button
            #searchToggle
            type="button"
-            class="immersive-header__search-toggle"
+            class="ih-search-toggle"
            aria-label="Buscar productos"
            aria-controls="immersive-search"
            [attr.aria-expanded]="isSearchOpen()"
@@ -93,7 +93,7 @@
          />
        }

-        <div class="immersive-header__account">
+        <div class="ih-account">
          <app-icon-button
            variant="user"
            ariaLabel="Mi cuenta"
@@ -109,7 +109,7 @@
              (logoutClick)="onLogoutClick()"
            />
          } @else if (isUserDropdownOpen()) {
-            <div id="immersive-user-dropdown" class="immersive-header__guest-menu">
+            <div id="immersive-user-dropdown" class="ih-guest-menu">
              <button type="button" (click)="isUserDropdownOpen.set(false); loginClick.emit()">
                Iniciar sesión
              </button>
@@ -121,7 +121,7 @@

    @if (checkoutRemainingTime(); as remainingTime) {
      <div
-        class="immersive-header__timer container-xl px-3 px-md-4"
+        class="ih-timer container-xl px-3 px-md-4"
        role="timer"
        aria-label="Tiempo restante de compra"
      >
@@ -131,10 +131,10 @@

    @if (showHero()) {
      @if (heroContent(); as content) {
-        <div class="immersive-header__copy">
-          <p class="immersive-header__eyebrow">{{ content.eyebrow }}</p>
-          <h1 class="immersive-header__title">{{ content.title }}</h1>
-          <p class="immersive-header__description">{{ content.description }}</p>
+        <div class="ih-copy">
+          <p class="ih-eyebrow">{{ content.eyebrow }}</p>
+          <h1 class="ih-title">{{ content.title }}</h1>
+          <p class="ih-description">{{ content.description }}</p>
        </div>
      }

--- a/src/app/core/layout/store-layout/store-header/immersive-store-header.component.scss
+++ b/src/app/core/layout/store-layout/store-header/immersive-store-header.component.scss
@@ -11,7 +11,7 @@
  background: #f5f5f5;
 }

-.immersive-header__content {
+.ih-content {
  grid-area: 1 / 1;
  position: relative;
  z-index: 1;
@@ -20,7 +20,7 @@
  padding-bottom: clamp(1rem, 2vw, 1.5rem);
 }

-.immersive-header__backdrop {
+.ih-backdrop {
  grid-area: 1 / 1;
  position: relative;
  z-index: 0;
@@ -28,7 +28,7 @@
  background-color: #101010;
 }

-.immersive-header__backdrop-image {
+.ih-backdrop-image {
  position: absolute;
  inset: 0;
  z-index: 0;
@@ -39,8 +39,8 @@
  object-position: center 42%;
 }

-.immersive-header__backdrop::before,
-.immersive-header__backdrop::after {
+.ih-backdrop::before,
+.ih-backdrop::after {
  content: '';
  position: absolute;
  inset: 0;
@@ -48,15 +48,15 @@
  pointer-events: none;
 }

-.immersive-header__backdrop::before {
+.ih-backdrop::before {
  background: rgba(0, 0, 0, 0.7);
 }

-.immersive-header__backdrop::after {
+.ih-backdrop::after {
  background: linear-gradient(180deg, rgba(0, 0, 0, 0.48) 0%, rgba(0, 0, 0, 0.4) 47%, #f5f5f5 100%);
 }

-.immersive-header__nav {
+.ih-nav {
  position: relative;
  // Keep the whole navigation stacking context above the hero copy and carousel.
  // Otherwise those later siblings can sit on top of the dropdown and capture
@@ -69,28 +69,28 @@
  min-height: 2.25rem;
 }

-.immersive-header__brand {
+.ih-brand {
  display: inline-flex;
  flex: none;
  align-items: center;
 }
-.immersive-header__brand img {
+.ih-brand img {
  display: block;
  width: clamp(105px, 13vw, 145px);
  max-height: 2.25rem;
  object-fit: contain;
 }
-.immersive-header__actions {
+.ih-actions {
  display: flex;
  align-items: center;
  gap: clamp(0.35rem, 1vw, 1rem);
 }

-.immersive-header__search {
+.ih-search {
  position: relative;
  width: clamp(115px, 17vw, 205px);
 }
-.immersive-header__search-error {
+.ih-search-error {
  position: absolute;
  top: calc(100% + 0.25rem);
  right: 0;
@@ -99,52 +99,52 @@
  font-size: 0.7rem;
 }

-:host ::ng-deep .immersive-header__tickets {
+:host ::ng-deep .ih-tickets {
  width: auto;
 }
-.immersive-header__account {
+.ih-account {
  position: relative;
  // The nav becomes display: contents on mobile, so it cannot provide the
  // stacking context there. Keep the dropdown trigger above following rows.
  z-index: 10;
  display: flex;
 }
-:host ::ng-deep .immersive-header__actions .icon-btn,
-:host ::ng-deep .immersive-header__actions .cart-icon {
+:host ::ng-deep .ih-actions .icon-btn,
+:host ::ng-deep .ih-actions .cart-icon {
  color: var(--tenant-primary, #ff7006);
 }

-.immersive-header__copy {
+.ih-copy {
  position: relative;
  z-index: 1;
  margin: clamp(2.1rem, 5vw, 3.25rem) auto 0;
  padding-inline: 1rem;
  text-align: center;
 }
-.immersive-header__eyebrow,
-.immersive-header__title {
+.ih-eyebrow,
+.ih-title {
  font-family: 'Smooch', cursive;
  font-weight: 400;
  text-shadow: 0 2px 12px rgba(0, 0, 0, 0.55);
 }
-.immersive-header__eyebrow {
+.ih-eyebrow {
  margin: 0;
  font-size: 50px;
  line-height: 1;
 }
-.immersive-header__title {
+.ih-title {
  margin: -0.15rem 0 0;
  font-size: 140px;
  line-height: 0.7;
 }
-.immersive-header__description {
+.ih-description {
  margin: 30px 0 0;
  font-family: 'Gotham Book', sans-serif;
  font-size: 18px;
  font-weight: 300;
  text-shadow: 0 1px 5px rgba(0, 0, 0, 0.6);
 }
-.immersive-header__timer {
+.ih-timer {
  position: relative;
  z-index: 1;
  margin-top: 0.75rem;
@@ -163,27 +163,27 @@
  color: #666666;
  background: transparent;

-  .immersive-header__content {
+  .ih-content {
    padding: 35px 0 36px;
  }

-  .immersive-header__nav {
+  .ih-nav {
    flex-wrap: nowrap;
    gap: 0.75rem;
    min-height: 44px;
  }

-  .immersive-header__brand img {
+  .ih-brand img {
    width: clamp(92px, 24vw, 145px);
  }

-  .immersive-header__actions {
+  .ih-actions {
    flex: 0 1 auto;
    gap: 0.375rem;
  }
 }

-.immersive-header__search-toggle {
+.ih-search-toggle {
  display: none;
  flex: none;
  align-items: center;
@@ -203,7 +203,7 @@
  }
 }

-.immersive-header__guest-menu {
+.ih-guest-menu {
  position: absolute;
  top: calc(100% + 0.5rem);
  right: 0;
@@ -224,7 +224,7 @@
 }

 :host ::ng-deep .immersive-header--compact {
-  .immersive-header__tickets {
+  .ih-tickets {
    flex: none;
    width: auto;

@@ -234,58 +234,58 @@
    }
  }

-  .immersive-header__account .icon-btn {
+  .ih-account .icon-btn {
    width: 32px;
    height: 44px;
    color: #888;
  }
 }
-.immersive-header--compact .immersive-header__backdrop {
+.immersive-header--compact .ih-backdrop {
  display: none;
 }

-:host ::ng-deep .immersive-header--compact .immersive-header__actions .icon-btn,
-:host ::ng-deep .immersive-header--compact .immersive-header__actions .cart-icon {
+:host ::ng-deep .immersive-header--compact .ih-actions .icon-btn,
+:host ::ng-deep .immersive-header--compact .ih-actions .cart-icon {
  color: #888;
 }

@media (max-width: 359.98px) {
-  .immersive-header--compact .immersive-header__nav {
+  .immersive-header--compact .ih-nav {
    padding-inline: 1rem !important;
    gap: 0.375rem;
  }

-  .immersive-header--compact .immersive-header__actions {
+  .immersive-header--compact .ih-actions {
    gap: 0.125rem;
  }
 }

@media (max-width: 600px) {
-  .immersive-header__nav {
+  .ih-nav {
    flex-wrap: wrap;
    gap: 0.5rem;
  }
-  .immersive-header__actions {
+  .ih-actions {
    flex: 1;
    justify-content: flex-end;
  }
-  .immersive-header__search {
+  .ih-search {
    width: clamp(105px, 38vw, 180px);
  }
-  .immersive-header__copy {
+  .ih-copy {
    margin-top: 3rem;
  }
-  .immersive-header__eyebrow {
+  .ih-eyebrow {
    font-size: clamp(2rem, 8vw, 50px);
  }
-  .immersive-header__title {
+  .ih-title {
    font-size: clamp(3rem, 12vw, 5rem);
  }
 }

@media (max-width: 767.98px) {
  .immersive-header--compact {
-    .immersive-header__search {
+    .ih-search {
      position: absolute;
      top: calc(100% + 0.5rem);
      right: 8vw;
@@ -297,17 +297,17 @@
      box-shadow: 0 4px 16px #0002;
    }

-    .immersive-header__search--open {
+    .ih-search--open {
      display: block;
    }

-    .immersive-header__search-error {
+    .ih-search-error {
      position: static;
      display: block;
      margin-top: 0.25rem;
    }

-    .immersive-header__search-toggle {
+    .ih-search-toggle {
      display: inline-flex;
    }
  }
@@ -315,7 +315,7 @@
  .immersive-header:not(.immersive-header--compact) {
    min-height: 0;

-    .immersive-header__content {
+    .ih-content {
      display: grid;
      grid-template-columns: minmax(0, 1fr) auto auto;
      align-items: center;
@@ -324,17 +324,17 @@
    }

    // Keep one search form while placing it below the hero copy on mobile.
-    .immersive-header__nav,
-    .immersive-header__actions {
+    .ih-nav,
+    .ih-actions {
      display: contents;
    }

-    .immersive-header__brand {
+    .ih-brand {
      grid-area: 1 / 1;
      margin-left: 8vw;
    }

-    .immersive-header__brand img {
+    .ih-brand img {
      width: clamp(105px, 34vw, 145px);
    }

@@ -342,30 +342,30 @@
      grid-area: 1 / 2;
    }

-    .immersive-header__account {
+    .ih-account {
      grid-area: 1 / 3;
      margin-right: calc(8vw - 0.75rem);
    }

-    .immersive-header__timer {
+    .ih-timer {
      grid-area: 2 / 1 / auto / -1;
    }

-    .immersive-header__copy {
+    .ih-copy {
      grid-area: 3 / 1 / auto / -1;
      margin: 1.25rem 0 0;
    }

-    .immersive-header__eyebrow {
+    .ih-eyebrow {
      font-size: clamp(18px, 5.5vw, 24px);
    }

-    .immersive-header__title {
+    .ih-title {
      font-size: clamp(40px, 12vw, 56px);
      line-height: 0.7;
    }

-    .immersive-header__description {
+    .ih-description {
      margin-top: 30px;
      font-size: clamp(11px, 3vw, 14px);
      line-height: 1.5;
@@ -373,14 +373,14 @@
      text-shadow: none;
    }

-    .immersive-header__search {
+    .ih-search {
      grid-area: 4 / 1 / auto / -1;
      width: 74%;
      max-width: 28rem;
      margin: 0.5rem auto 0;
    }

-    .immersive-header__search-error {
+    .ih-search-error {
      right: auto;
      left: 0;
      width: 100%;
@@ -395,11 +395,11 @@
      --carousel-offset: -9vw;
    }

-    .immersive-header__backdrop-image {
+    .ih-backdrop-image {
      object-position: center 42%;
    }

-    .immersive-header__backdrop {
+    .ih-backdrop {
      // Keep the fade tied to the hero, independent of the content below it.
      align-self: start;
      height: 12.5rem;
@@ -407,7 +407,7 @@
      mask-image: linear-gradient(#000 calc(100% - 8px), transparent 100%);
    }

-    .immersive-header__backdrop::after {
+    .ih-backdrop::after {
      background: linear-gradient(
        180deg,
        rgba(0, 0, 0, 0.25) 0%,
@@ -420,7 +420,7 @@
  }

  :host ::ng-deep .immersive-header:not(.immersive-header--compact) {
-    .immersive-header__search .form-control {
+    .ih-search .form-control {
      min-height: 36px;
      padding: 0.25rem 2.5rem 0.25rem 0.75rem;
      border: 1px solid #888;
@@ -439,12 +439,12 @@
      }
    }

-    .immersive-header__search .input-search-icon {
+    .ih-search .input-search-icon {
      color: #666;
      right: 0.75rem;
    }

-    .immersive-header__account .icon-btn {
+    .ih-account .icon-btn {
      min-width: 44px;
      min-height: 44px;

--- a/src/app/core/layout/store-layout/store-layout.component.spec.ts
+++ b/src/app/core/layout/store-layout/store-layout.component.spec.ts
@@ -274,21 +274,21 @@ describe('StoreLayoutComponent', () => {
    expect(header.componentInstance.headerType()).toBe('immersive');
    expect(element.querySelector('app-standard-store-header')).toBeNull();
    expect(element.querySelector('app-immersive-store-header')).not.toBeNull();
-    expect(element.querySelector('.immersive-header__eyebrow')?.textContent).toContain(
+    expect(element.querySelector('.ih-eyebrow')?.textContent).toContain(
      'Encendé tu',
    );
-    expect(element.querySelector('.immersive-header__title')?.textContent).toContain('experiencia');
-    expect(element.querySelector('.immersive-header__description')?.textContent).toContain(
+    expect(element.querySelector('.ih-title')?.textContent).toContain('experiencia');
+    expect(element.querySelector('.ih-description')?.textContent).toContain(
      'Reservá tu entrada',
    );
    const backdropImage = element.querySelector<HTMLImageElement>(
-      '.immersive-header__backdrop-image',
+      '.ih-backdrop-image',
    );
    expect(backdropImage?.getAttribute('src')).toBe(tenant.header_bg_image);
    expect(backdropImage?.getAttribute('loading')).toBe('eager');
    expect(backdropImage?.getAttribute('fetchpriority')).toBe('high');
    expect(backdropImage?.getAttribute('decoding')).toBe('async');
-    const headerLogo = element.querySelector<HTMLImageElement>('.immersive-header__brand img');
+    const headerLogo = element.querySelector<HTMLImageElement>('.ih-brand img');
    expect(headerLogo?.getAttribute('loading')).toBe('eager');
    expect(headerLogo?.getAttribute('fetchpriority')).toBe('high');
    expect(headerLogo?.getAttribute('decoding')).toBe('async');
--- a/src/app/features/store/components/product-carousel/product-carousel.component.html
+++ b/src/app/features/store/components/product-carousel/product-carousel.component.html
@@ -62,13 +62,7 @@
          (click)="selectImage(idx)"
          [attr.aria-label]="'Select image ' + (idx + 1)"
        >
-          <img
-            [src]="image"
-            alt="Product thumbnail"
-            loading="lazy"
-            decoding="async"
-            class="w-100 h-100 object-fit-cover"
-          />
+          <img [src]="image" alt="Product thumbnail" class="w-100 h-100 object-fit-cover" />
        </button>
      }
    </div>
--- a/src/app/features/store/pages/store-home-page/store-home-page.component.html
+++ b/src/app/features/store/pages/store-home-page/store-home-page.component.html
@@ -23,49 +23,24 @@
  </p>
 } @else {
  @for (group of catalog(); track group.id; let first = $first) {
-    @if (first) {
-      <ng-container
-        [ngTemplateOutlet]="catalogGroup"
-        [ngTemplateOutletContext]="{ $implicit: group, prioritizeFirstImage: true }"
+    <app-store-section [attr.id]="group.code" [title]="group.title">
+      <app-product-list
+        [layout]="group.layout"
+        [groupLayout]="group.group_layout"
+        [items]="group.items"
+        [loading]="isGroupLoading(group.id)"
+        [loadImages]="!hasMainCarouselImages() || mainCarouselReady()"
+        [prioritizeFirstImage]="first && !hasMainCarouselImages()"
+        [unavailableVariantIds]="unavailableVariantIds()"
+        [savingProductIds]="savingProductIds()"
+        (buy)="onBuyProduct($event)"
+        (addToCart)="onAddToCart($event)"
+        (pageChange)="onPageChange(group.id, $event)"
      />
-    } @else {
-      @defer (on viewport; prefetch on idle) {
-        <ng-container
-          [ngTemplateOutlet]="catalogGroup"
-          [ngTemplateOutletContext]="{ $implicit: group, prioritizeFirstImage: false }"
-        />
-      } @placeholder {
-        <section
-          class="store-home__deferred-group-placeholder"
-          [attr.aria-label]="group.title"
-        ></section>
-      }
-    }
+    </app-store-section>
  }
 }

-<ng-template
-  #catalogGroup
-  let-group
-  let-prioritizeFirstImage="prioritizeFirstImage"
->
-  <app-store-section [attr.id]="group.code" [title]="group.title">
-    <app-product-list
-      [layout]="group.layout"
-      [groupLayout]="group.group_layout"
-      [items]="group.items"
-      [loading]="isGroupLoading(group.id)"
-      [loadImages]="!hasMainCarouselImages() || mainCarouselReady()"
-      [prioritizeFirstImage]="prioritizeFirstImage && !hasMainCarouselImages()"
-      [unavailableVariantIds]="unavailableVariantIds()"
-      [savingProductIds]="savingProductIds()"
-      (buy)="onBuyProduct($event)"
-      (addToCart)="onAddToCart($event)"
-      (pageChange)="onPageChange(group.id, $event)"
-    />
-  </app-store-section>
-</ng-template>
-
@if (additionalInfo(); as content) {
  <app-store-section class="store-home__additional-info" title="Información adicional">
    <div class="store-home__additional-info-content" [innerHTML]="content"></div>
--- a/src/app/features/store/pages/store-home-page/store-home-page.component.scss
+++ b/src/app/features/store/pages/store-home-page/store-home-page.component.scss
@@ -15,13 +15,6 @@ app-store-section[id] {
  scroll-margin-top: 8rem;
 }

-.store-home__deferred-group-placeholder {
-  display: block;
-  min-height: clamp(24rem, 42vw, 34rem);
-  content-visibility: auto;
-  contain-intrinsic-size: auto 30rem;
-}
-
 :host > .store-home__additional-info:not(:first-child) {
  margin-top: clamp(3rem, 6vw, 5rem);
 }
--- a/src/app/features/store/pages/store-home-page/store-home-page.component.ts
+++ b/src/app/features/store/pages/store-home-page/store-home-page.component.ts
@@ -8,7 +8,6 @@ import {
  inject,
  signal,
 } from '@angular/core';
-import { NgTemplateOutlet } from '@angular/common';
 import { HttpErrorResponse } from '@angular/common/http';
 import { ActivatedRoute, Router } from '@angular/router';
 import { finalize, Subscription } from 'rxjs';
@@ -45,7 +44,6 @@ import {
    ProductListComponent,
    HeroBannerComponent,
    MainCarouselComponent,
-    NgTemplateOutlet,
  ],
  templateUrl: './store-home-page.component.html',
  styleUrl: './store-home-page.component.scss',
--- a/src/app/shared/components/carousel/carousel.component.html
+++ b/src/app/shared/components/carousel/carousel.component.html
@@ -1,10 +1,4 @@
 <section class="carousel" role="region" [attr.aria-label]="ariaLabel()">
-  <span
-    #sizeProbe
-    class="carousel__size-probe"
-    aria-hidden="true"
-    [style.width]="itemBasis()"
-  ></span>
  <div class="carousel__frame" [class.carousel__frame--with-controls]="hasMultipleItems()">
    <div
      #viewport
--- a/src/app/shared/components/carousel/carousel.component.scss
+++ b/src/app/shared/components/carousel/carousel.component.scss
@@ -4,17 +4,8 @@
 }

 .carousel {
-  position: relative;
  width: 100%;

-  &__size-probe {
-    position: absolute;
-    height: 0;
-    visibility: hidden;
-    pointer-events: none;
-    contain: strict;
-  }
-
  &__frame {
    position: relative;
    display: grid;
--- a/src/app/shared/components/carousel/carousel.component.spec.ts
+++ b/src/app/shared/components/carousel/carousel.component.spec.ts
@@ -1,6 +1,6 @@
 import { Component, signal, TemplateRef, viewChild } from '@angular/core';
 import { ComponentFixture, TestBed } from '@angular/core/testing';
-import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
+import { beforeEach, describe, expect, it, vi } from 'vitest';
 import { CarouselComponent, CarouselItemContext } from './carousel.component';

 interface TestItem {
@@ -16,7 +16,7 @@ interface TestItem {
    </ng-template>

    <app-carousel
-      [items]="items()"
+      [items]="items"
      [itemTemplate]="itemTemplate"
      [scrollStep]="120"
      [gap]="16"
@@ -28,11 +28,11 @@ interface TestItem {
 class TestHostComponent {
  readonly circular = signal(false);

-  readonly items = signal<TestItem[]>([
+  readonly items: TestItem[] = [
    { id: 1, label: 'Primero' },
    { id: 2, label: 'Segundo' },
    { id: 3, label: 'Tercero' },
-  ]);
+  ];

  readonly carousel = viewChild.required(CarouselComponent<TestItem>);
  readonly itemTemplate =
@@ -41,47 +41,8 @@ class TestHostComponent {

 describe('CarouselComponent', () => {
  let fixture: ComponentFixture<TestHostComponent>;
-  let frames: Map<number, FrameRequestCallback>;
-  let resize: () => void;
-  let disconnect: ReturnType<typeof vi.fn>;
-  const flushLayout = () => {
-    const pending = [...frames.values()];
-    frames.clear();
-    pending.forEach((callback) => callback(0));
-    fixture.detectChanges();
-  };
-  const setDimensions = (width: number, basis: number) => {
-    const host = fixture.nativeElement.querySelector('app-carousel');
-    Object.defineProperty(host, 'clientWidth', { configurable: true, value: width });
-    const probe = fixture.nativeElement.querySelector('.carousel__size-probe');
-    probe.getBoundingClientRect = () => ({ width: basis }) as DOMRect;
-  };

  beforeEach(async () => {
-    frames = new Map();
-    let frameId = 0;
-    vi.stubGlobal(
-      'requestAnimationFrame',
-      vi.fn((callback: FrameRequestCallback) => {
-        frames.set(++frameId, callback);
-        return frameId;
-      }),
-    );
-    vi.stubGlobal(
-      'cancelAnimationFrame',
-      vi.fn((id: number) => frames.delete(id)),
-    );
-    disconnect = vi.fn();
-    vi.stubGlobal(
-      'ResizeObserver',
-      class {
-        constructor(callback: () => void) {
-          resize = callback;
-        }
-        observe() {}
-        disconnect = disconnect;
-      },
-    );
    await TestBed.configureTestingModule({
      imports: [TestHostComponent],
    }).compileComponents();
@@ -90,75 +51,6 @@ describe('CarouselComponent', () => {
    fixture.detectChanges();
  });

-  afterEach(() => {
-    fixture.destroy();
-    vi.unstubAllGlobals();
-  });
-
-  it('agrupa las mediciones y no se realimenta al renderizar o navegar', async () => {
-    fixture.componentInstance.circular.set(true);
-    setDimensions(650, 200);
-    fixture.detectChanges();
-    resize();
-    resize();
-    expect(frames.size).toBe(1);
-    flushLayout();
-    await Promise.resolve();
-    expect(frames.size).toBe(0);
-    const carousel = fixture.componentInstance.carousel();
-    expect(carousel['itemsPerPage']()).toBe(2);
-    carousel.next();
-    fixture.detectChanges();
-    const track = fixture.nativeElement.querySelector('.carousel__track');
-    track.dispatchEvent(new TransitionEvent('transitionend', { propertyName: 'transform' }));
-    fixture.detectChanges();
-    await Promise.resolve();
-    resize(); // Same width, even if the new cards have a different height.
-    expect(frames.size).toBe(0);
-    expect(carousel['currentPage']()).toBe(1);
-    expect(carousel['itemWidth']()).toBe(269);
-  });
-
-  it('recalcula al cambiar el ancho y la base responsive de las tarjetas', () => {
-    setDimensions(650, 200);
-    resize();
-    flushLayout();
-    setDimensions(350, 200);
-    resize();
-    flushLayout();
-    expect(fixture.componentInstance.carousel()['itemsPerPage']()).toBe(1);
-    setDimensions(350, 100);
-    resize();
-    flushLayout();
-    expect(fixture.componentInstance.carousel()['itemsPerPage']()).toBe(2);
-    expect(frames.size).toBe(0);
-  });
-
-  it('actualiza productos y admite una lista vacía sin anchos inválidos', () => {
-    setDimensions(650, 200);
-    flushLayout();
-    fixture.componentInstance.items.set([]);
-    fixture.detectChanges();
-    expect(frames.size).toBe(1);
-    flushLayout();
-    const carousel = fixture.componentInstance.carousel();
-    expect(carousel['itemWidth']()).toBeNull();
-    expect(carousel['hasOverflow']()).toBe(false);
-    fixture.componentInstance.items.set([{ id: 4, label: 'Nuevo' }]);
-    fixture.detectChanges();
-    flushLayout();
-    expect(carousel['itemWidth']()).toBe(650);
-    expect(fixture.nativeElement.querySelectorAll('article')).toHaveLength(1);
-    expect(frames.size).toBe(0);
-  });
-
-  it('cancela mediciones pendientes y desconecta el observador al destruirse', () => {
-    expect(frames.size).toBe(1);
-    fixture.destroy();
-    expect(frames.size).toBe(0);
-    expect(disconnect).toHaveBeenCalledOnce();
-  });
-
  it('renderiza el template provisto para cada elemento', () => {
    const articles = fixture.nativeElement.querySelectorAll('article');

@@ -167,17 +59,6 @@ describe('CarouselComponent', () => {
    expect(articles[1].getAttribute('data-index')).toBe('1');
  });

-  it('permite inspeccionar las páginas antes de medir el viewport sin generar listas enormes', () => {
-    const carousel = fixture.componentInstance.carousel();
-    fixture.componentInstance.circular.set(true);
-    carousel['itemsPerPage'].set(Number.MAX_SAFE_INTEGER);
-
-    expect(carousel['circularPages']()).toEqual([]);
-    expect(carousel['createPage']('current', 0).items.map((entry) => entry.item.id)).toEqual([
-      1, 2, 3,
-    ]);
-  });
-
  it('expone una región accesible y controles con etiquetas', () => {
    const region = fixture.nativeElement.querySelector('[role="region"]');
    const buttons = fixture.nativeElement.querySelectorAll('button');
@@ -218,7 +99,7 @@ describe('CarouselComponent', () => {

  it('usa todo el ancho y distribuye el espacio sobrante entre elementos completos', () => {
    const host = fixture.nativeElement.querySelector('app-carousel') as HTMLElement;
-    const firstItem = fixture.nativeElement.querySelector('.carousel__size-probe') as HTMLElement;
+    const firstItem = fixture.nativeElement.querySelector('.carousel__item') as HTMLElement;

    Object.defineProperty(host, 'clientWidth', {
      configurable: true,
@@ -231,7 +112,7 @@ describe('CarouselComponent', () => {
        }) as DOMRect,
    );

-    flushLayout();
+    fixture.componentInstance.carousel().ngAfterViewInit();
    fixture.detectChanges();

    const frame = fixture.nativeElement.querySelector('.carousel__frame') as HTMLElement;
@@ -247,7 +128,7 @@ describe('CarouselComponent', () => {
    fixture.detectChanges();

    const host = fixture.nativeElement.querySelector('app-carousel') as HTMLElement;
-    const firstItem = fixture.nativeElement.querySelector('.carousel__size-probe') as HTMLElement;
+    const firstItem = fixture.nativeElement.querySelector('.carousel__item') as HTMLElement;

    Object.defineProperty(host, 'clientWidth', {
      configurable: true,
@@ -260,7 +141,7 @@ describe('CarouselComponent', () => {
        }) as DOMRect,
    );

-    flushLayout();
+    fixture.componentInstance.carousel().ngAfterViewInit();
    fixture.detectChanges();

    const pages = fixture.nativeElement.querySelectorAll('.carousel__page');
--- a/src/app/shared/components/carousel/carousel.component.ts
+++ b/src/app/shared/components/carousel/carousel.component.ts
@@ -1,6 +1,5 @@
 import {
  AfterViewInit,
-  afterRenderEffect,
  ChangeDetectionStrategy,
  Component,
  DestroyRef,
@@ -11,7 +10,6 @@ import {
  inject,
  input,
  signal,
-  untracked,
  viewChild,
 } from '@angular/core';
 import { isPlatformBrowser, NgTemplateOutlet } from '@angular/common';
@@ -47,15 +45,12 @@ export class CarouselComponent<T> implements AfterViewInit {
  private readonly platformId = inject(PLATFORM_ID);
  private readonly host = inject<ElementRef<HTMLElement>>(ElementRef);
  private readonly viewport = viewChild.required<ElementRef<HTMLElement>>('viewport');
-  private readonly sizeProbe = viewChild.required<ElementRef<HTMLElement>>('sizeProbe');
-  private layoutFrame: number | null = null;

  readonly items = input.required<readonly T[]>();
  readonly itemTemplate = input.required<TemplateRef<CarouselItemContext<T>>>();
  readonly trackBy = input<CarouselTrackBy<T>>((index) => index);
  readonly scrollStep = input<number | null>(null);
  readonly gap = input(0);
-  readonly itemBasis = input('16rem');
  readonly circular = input(false);
  readonly ariaLabel = input('Carrusel');
  readonly previousLabel = input('Mostrar elementos anteriores');
@@ -75,10 +70,6 @@ export class CarouselComponent<T> implements AfterViewInit {
  );
  protected readonly isCircularLayout = computed(() => this.circular() && this.pageCount() > 1);
  protected readonly circularPages = computed<CarouselPage<T>[]>(() => {
-    if (!this.isCircularLayout()) {
-      return [];
-    }
-
    const pageCount = this.pageCount();
    const currentPage = this.normalizePage(this.currentPage(), pageCount);

@@ -95,51 +86,27 @@ export class CarouselComponent<T> implements AfterViewInit {
    () => !this.hasOverflow() || (!this.circular() && !this.canScrollNext()),
  );

-  constructor() {
-    afterRenderEffect(() => {
-      this.items();
-      this.gap();
-      this.circular();
-      this.itemBasis();
-      this.itemTemplate();
-      // Layout outputs must never become dependencies of this effect.
-      untracked(() => this.scheduleLayout());
-    });
-  }
-
-  private scheduleLayout(): void {
-    if (this.destroyRef.destroyed || this.layoutFrame !== null) return;
-    this.layoutFrame = requestAnimationFrame(() => {
-      this.layoutFrame = null;
-      this.refreshLayout();
-    });
-  }
-
  ngAfterViewInit(): void {
    if (!isPlatformBrowser(this.platformId)) {
      return;
    }

-    let previousWidth = -1;
-    let previousBasis = -1;
+    const viewport = this.viewport().nativeElement;
+    this.refreshLayout();
+
    const resizeObserver =
-      typeof ResizeObserver === 'undefined'
-        ? null
-        : new ResizeObserver(() => {
-            const width = this.host.nativeElement.clientWidth;
-            const basis = this.sizeProbe().nativeElement.getBoundingClientRect().width;
-            // Image loads and page rotation can change height, but not the layout inputs.
-            if (width === previousWidth && basis === previousBasis) return;
-            previousWidth = width;
-            previousBasis = basis;
-            this.scheduleLayout();
-          });
+      typeof ResizeObserver === 'undefined' ? null : new ResizeObserver(() => this.refreshLayout());
    resizeObserver?.observe(this.host.nativeElement);
-    resizeObserver?.observe(this.sizeProbe().nativeElement);
+
+    const mutationObserver =
+      typeof MutationObserver === 'undefined'
+        ? null
+        : new MutationObserver(() => this.refreshLayout());
+    mutationObserver?.observe(viewport, { childList: true, subtree: true });

    this.destroyRef.onDestroy(() => {
      resizeObserver?.disconnect();
-      if (this.layoutFrame !== null) cancelAnimationFrame(this.layoutFrame);
+      mutationObserver?.disconnect();
    });
  }

@@ -251,32 +218,24 @@ export class CarouselComponent<T> implements AfterViewInit {
  }

  private refreshLayout(): void {
-    const itemCount = this.items().length;
+    const viewport = this.viewport().nativeElement;
+    const firstItem = viewport.querySelector<HTMLElement>('.carousel__item');
    const availableWidth = this.host.nativeElement.clientWidth;
    const availableViewportWidth = Math.max(
      availableWidth - (this.hasMultipleItems() ? CarouselComponent.CONTROL_SLOT_WIDTH * 2 : 0),
      0,
    );

-    if (itemCount === 0) {
-      this.itemsPerPage.set(1);
-      this.currentPage.set(0);
-      this.transitionDirection.set(0);
-      this.itemWidth.set(null);
-      this.hasOverflow.set(false);
-      this.canScrollPrevious.set(false);
-      this.canScrollNext.set(false);
-      return;
-    }
-
-    if (availableViewportWidth <= 0) {
+    if (!firstItem || availableViewportWidth <= 0) {
      this.itemWidth.set(null);
      this.refreshNavigation();
      return;
    }

-    // Measure a stable CSS basis, never a card whose content/width we just changed.
-    const naturalItemWidth = this.sizeProbe().nativeElement.getBoundingClientRect().width;
+    const assignedWidth = firstItem.style.width;
+    firstItem.style.width = '';
+    const naturalItemWidth = firstItem.getBoundingClientRect().width;
+    firstItem.style.width = assignedWidth;

    if (naturalItemWidth <= 0) {
      this.itemWidth.set(null);
@@ -285,9 +244,9 @@ export class CarouselComponent<T> implements AfterViewInit {
    }

    const gap = this.resolvedGap();
-    const fittingItems = Math.min(
-      itemCount,
-      Math.max(1, Math.floor((availableViewportWidth + gap) / (naturalItemWidth + gap))),
+    const fittingItems = Math.max(
+      1,
+      Math.floor((availableViewportWidth + gap) / (naturalItemWidth + gap)),
    );
    const visibleItems = Math.min(fittingItems, this.items().length);
    const distributedItemWidth =
@@ -299,12 +258,7 @@ export class CarouselComponent<T> implements AfterViewInit {
      this.transitionDirection.set(0);
    }

-    if (this.itemWidth() !== distributedItemWidth) this.itemWidth.set(distributedItemWidth);
-    const normalizedPage = this.normalizePage(this.currentPage(), this.pageCount());
-    if (normalizedPage !== this.currentPage()) {
-      this.currentPage.set(normalizedPage);
-      this.transitionDirection.set(0);
-    }
+    this.itemWidth.set(distributedItemWidth);
    if (this.circular()) {
      const hasOverflow = this.items().length > fittingItems;
      this.hasOverflow.set(hasOverflow);
@@ -316,20 +270,14 @@ export class CarouselComponent<T> implements AfterViewInit {
  }

  private createPage(slot: CarouselPage<T>['slot'], page: number): CarouselPage<T> {
-    const sourceItems = this.items();
-    // Layout has not necessarily been measured when a computed value is inspected.
-    const pageSize = Math.min(this.itemsPerPage(), sourceItems.length);
-    if (pageSize === 0) {
-      return { slot, items: [] };
-    }
    const pageCount = this.pageCount();
    const normalizedPage = this.normalizePage(page, pageCount);
-    const start = normalizedPage * pageSize;
+    const start = normalizedPage * this.itemsPerPage();
    const items: CarouselPageItem<T>[] = [];

-    for (let offset = 0; offset < pageSize; offset += 1) {
-      const index = (start + offset) % sourceItems.length;
-      items.push({ item: sourceItems[index], index });
+    for (let offset = 0; offset < this.itemsPerPage(); offset += 1) {
+      const index = (start + offset) % this.items().length;
+      items.push({ item: this.items()[index], index });
    }

    return { slot, items };
--- a/src/app/shared/components/product-list/product-list.component.html
+++ b/src/app/shared/components/product-list/product-list.component.html
@@ -67,9 +67,6 @@
    [itemTemplate]="productItem"
    [trackBy]="trackProduct"
    [gap]="24"
-    [itemBasis]="
-      effectiveLayout() === 'row' ? 'clamp(18rem, 70vw, 32rem)' : 'clamp(15rem, 30vw, 18rem)'
-    "
    [circular]="true"
    ariaLabel="Productos destacados"
  />
Author	SHA1	Message	Date
ncoronel	f70b93922e	Merge branch 'homologacion'	2026-09-24 17:00:14 -03:00
ncoronel	d49c78b993	refactor(store-header): update class names for consistency and readability	2026-09-24 16:59:55 -03:00
ncoronel	8ef00054c9	Merge pull request 'homologacion' (#9 ) from homologacion into main Reviewed-on: https://gitea.quo.ar/tbianchini/shopit-front/pulls/9	2026-09-24 19:54:56 +00:00
ncoronel	7247e8e694	Merge pull request 'homologacion' (#7 ) from homologacion into main Reviewed-on: https://gitea.quo.ar/tbianchini/shopit-front/pulls/7	2026-09-17 17:52:29 +00:00