feat(carousel): add size probe for responsive layout and optimize item width calculation

feat(carousel): enhance circular layout handling and optimize page creation logic
feat(store-home-page): implement lazy loading for product thumbnails and enhance deferred loading for sections
2026-09-24 15:42:44 -03:00 · 2026-09-24 15:20:57 -03:00 · 2026-09-24 14:12:30 -03:00
12 changed files with 367 additions and 138 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="ih-backdrop">
+  <div class="immersive-header__backdrop">
    @if (showHero() && backgroundImageUrl(); as backgroundImage) {
      <img
-        class="ih-backdrop-image"
+        class="immersive-header__backdrop-image"
        [src]="backgroundImage"
        alt=""
        loading="eager"
@@ -12,12 +12,12 @@
    }
  </div>

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

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

-        <div class="ih-account">
+        <div class="immersive-header__account">
          <app-icon-button
            variant="user"
            ariaLabel="Mi cuenta"
@@ -109,7 +109,7 @@
              (logoutClick)="onLogoutClick()"
            />
          } @else if (isUserDropdownOpen()) {
-            <div id="immersive-user-dropdown" class="ih-guest-menu">
+            <div id="immersive-user-dropdown" class="immersive-header__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="ih-timer container-xl px-3 px-md-4"
+        class="immersive-header__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="ih-copy">
-          <p class="ih-eyebrow">{{ content.eyebrow }}</p>
-          <h1 class="ih-title">{{ content.title }}</h1>
-          <p class="ih-description">{{ content.description }}</p>
+        <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>
      }

--- 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;
 }

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

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

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

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

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

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

-.ih-nav {
+.immersive-header__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;
 }

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

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

-:host ::ng-deep .ih-tickets {
+:host ::ng-deep .immersive-header__tickets {
  width: auto;
 }
-.ih-account {
+.immersive-header__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 .ih-actions .icon-btn,
-:host ::ng-deep .ih-actions .cart-icon {
+:host ::ng-deep .immersive-header__actions .icon-btn,
+:host ::ng-deep .immersive-header__actions .cart-icon {
  color: var(--tenant-primary, #ff7006);
 }

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

-    .ih-content {
+    .immersive-header__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.
-    .ih-nav,
-    .ih-actions {
+    .immersive-header__nav,
+    .immersive-header__actions {
      display: contents;
    }

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

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

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

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

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

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

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

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

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

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

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

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

-    .ih-backdrop {
+    .immersive-header__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%);
    }

-    .ih-backdrop::after {
+    .immersive-header__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) {
-    .ih-search .form-control {
+    .immersive-header__search .form-control {
      min-height: 36px;
      padding: 0.25rem 2.5rem 0.25rem 0.75rem;
      border: 1px solid #888;
@@ -439,12 +439,12 @@
      }
    }

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

-    .ih-account .icon-btn {
+    .immersive-header__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('.ih-eyebrow')?.textContent).toContain(
+    expect(element.querySelector('.immersive-header__eyebrow')?.textContent).toContain(
      'Encendé tu',
    );
-    expect(element.querySelector('.ih-title')?.textContent).toContain('experiencia');
-    expect(element.querySelector('.ih-description')?.textContent).toContain(
+    expect(element.querySelector('.immersive-header__title')?.textContent).toContain('experiencia');
+    expect(element.querySelector('.immersive-header__description')?.textContent).toContain(
      'Reservá tu entrada',
    );
    const backdropImage = element.querySelector<HTMLImageElement>(
-      '.ih-backdrop-image',
+      '.immersive-header__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>('.ih-brand img');
+    const headerLogo = element.querySelector<HTMLImageElement>('.immersive-header__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,7 +62,13 @@
          (click)="selectImage(idx)"
          [attr.aria-label]="'Select image ' + (idx + 1)"
        >
-          <img [src]="image" alt="Product thumbnail" class="w-100 h-100 object-fit-cover" />
+          <img
+            [src]="image"
+            alt="Product thumbnail"
+            loading="lazy"
+            decoding="async"
+            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,24 +23,49 @@
  </p>
 } @else {
  @for (group of catalog(); track group.id; let first = $first) {
-    <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)"
+    @if (first) {
+      <ng-container
+        [ngTemplateOutlet]="catalogGroup"
+        [ngTemplateOutletContext]="{ $implicit: group, prioritizeFirstImage: true }"
      />
-    </app-store-section>
+    } @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>
+      }
+    }
  }
 }

+<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,6 +15,13 @@ 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,6 +8,7 @@ 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';
@@ -44,6 +45,7 @@ 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,4 +1,10 @@
 <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,8 +4,17 @@
 }

 .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 { beforeEach, describe, expect, it, vi } from 'vitest';
+import { afterEach, 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: TestItem[] = [
+  readonly items = signal<TestItem[]>([
    { id: 1, label: 'Primero' },
    { id: 2, label: 'Segundo' },
    { id: 3, label: 'Tercero' },
-  ];
+  ]);

  readonly carousel = viewChild.required(CarouselComponent<TestItem>);
  readonly itemTemplate =
@@ -41,8 +41,47 @@ 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();
@@ -51,6 +90,75 @@ 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');

@@ -59,6 +167,17 @@ 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');
@@ -99,7 +218,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__item') as HTMLElement;
+    const firstItem = fixture.nativeElement.querySelector('.carousel__size-probe') as HTMLElement;

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

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

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

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

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

-    fixture.componentInstance.carousel().ngAfterViewInit();
+    flushLayout();
    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,5 +1,6 @@
 import {
  AfterViewInit,
+  afterRenderEffect,
  ChangeDetectionStrategy,
  Component,
  DestroyRef,
@@ -10,6 +11,7 @@ import {
  inject,
  input,
  signal,
+  untracked,
  viewChild,
 } from '@angular/core';
 import { isPlatformBrowser, NgTemplateOutlet } from '@angular/common';
@@ -45,12 +47,15 @@ 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');
@@ -70,6 +75,10 @@ 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);

@@ -86,27 +95,51 @@ 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;
    }

-    const viewport = this.viewport().nativeElement;
-    this.refreshLayout();
-
+    let previousWidth = -1;
+    let previousBasis = -1;
    const resizeObserver =
-      typeof ResizeObserver === 'undefined' ? null : new ResizeObserver(() => this.refreshLayout());
-    resizeObserver?.observe(this.host.nativeElement);
-
-    const mutationObserver =
-      typeof MutationObserver === 'undefined'
+      typeof ResizeObserver === 'undefined'
        ? null
-        : new MutationObserver(() => this.refreshLayout());
-    mutationObserver?.observe(viewport, { childList: true, subtree: true });
+        : 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();
+          });
+    resizeObserver?.observe(this.host.nativeElement);
+    resizeObserver?.observe(this.sizeProbe().nativeElement);

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

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

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

-    if (!firstItem || availableViewportWidth <= 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) {
      this.itemWidth.set(null);
      this.refreshNavigation();
      return;
    }

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

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

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

-    this.itemWidth.set(distributedItemWidth);
+    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);
+    }
    if (this.circular()) {
      const hasOverflow = this.items().length > fittingItems;
      this.hasOverflow.set(hasOverflow);
@@ -270,14 +316,20 @@ 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 * this.itemsPerPage();
+    const start = normalizedPage * pageSize;
    const items: CarouselPageItem<T>[] = [];

-    for (let offset = 0; offset < this.itemsPerPage(); offset += 1) {
-      const index = (start + offset) % this.items().length;
-      items.push({ item: this.items()[index], index });
+    for (let offset = 0; offset < pageSize; offset += 1) {
+      const index = (start + offset) % sourceItems.length;
+      items.push({ item: sourceItems[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,6 +67,9 @@
    [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	3558b6f575	feat(carousel): add size probe for responsive layout and optimize item width calculation	2026-09-24 15:42:44 -03:00
ncoronel	f4959e0250	feat(carousel): enhance circular layout handling and optimize page creation logic	2026-09-24 15:20:57 -03:00
ncoronel	ee1420801d	feat(store-home-page): implement lazy loading for product thumbnails and enhance deferred loading for sections	2026-09-24 14:12:30 -03:00