From db16e6630d34f3d1f8960910087c309cddbbfa05 Mon Sep 17 00:00:00 2001
From: Sudev Kiyada <sudev.kiyada@thomsonreuters.com>
Date: Fri, 9 Jan 2026 12:22:01 +0530
Subject: [PATCH] change prop names for clarity

---
 .../HorizontalScroller/Debug.svelte           | 17 ++--
 .../HorizontalScroller/HorizontalScroller.mdx |  4 +-
 .../HorizontalScroller.stories.svelte         |  4 +-
 .../HorizontalScroller.svelte                 | 94 ++++++++++---------
 .../demo/DemoSnippet.svelte                   |  2 +-
 .../demo/withScrollerBase.svelte              |  2 +-
 6 files changed, 63 insertions(+), 60 deletions(-)

diff --git a/src/components/HorizontalScroller/Debug.svelte b/src/components/HorizontalScroller/Debug.svelte
index c6b5be70..3a96e9e7 100644
--- a/src/components/HorizontalScroller/Debug.svelte
+++ b/src/components/HorizontalScroller/Debug.svelte
@@ -48,7 +48,7 @@
 
   let normalisedScrollProgress = $derived(
     map(
-      componentState.scrollProgress,
+      componentState.mappedProgress,
       componentState.clampStart ?? 0,
       componentState.clampEnd ?? 1,
       0,
@@ -58,7 +58,7 @@
 
   let normalisedProgress = $derived(
     map(
-      componentState.progress,
+      componentState.easedProgress,
       componentState.clampStart ?? 0,
       componentState.clampEnd ?? 1,
       0,
@@ -120,22 +120,21 @@
     </summary>
     <div class="state-debug">
       <!--  -->
-      <p>Raw progress:</p>
+      <p>Progress:</p>
       <div style="display: flex; flex-direction: column; gap: 4px;">
         <p class="state-value">
-          <span class="tag">{componentState.rawProgress}</span>
+          {componentState.progress}
         </p>
       </div>
       <!--  -->
-      <!--  -->
-      <p>Scroll progress:</p>
+      <p>Mapped progress:</p>
       <div style="display: flex; flex-direction: column; gap: 4px;">
         <p class="state-value progress-value">
           {@render triggerPoints()}
           <span
             class="progress-stop"
             style={`left: ${normalisedScrollProgress * 100}%; transform: translateX(-50%);`}
-            >{fmt.format(componentState.scrollProgress)}</span
+            >{fmt.format(componentState.mappedProgress)}</span
           >
           &nbsp;
         </p>
@@ -146,7 +145,7 @@
         </div>
       </div>
       <!--  -->
-      <p>Progress:</p>
+      <p>Eased Progress:</p>
       <div style="display: flex; flex-direction: column; gap: 4px;">
         <p class="state-value progress-value">
           {#if componentState.stops.length > 0}
@@ -159,7 +158,7 @@
           <span
             class="progress-stop"
             style={`left: ${normalisedProgress * 100}%; transform: translateX(-50%);`}
-            >{fmt.format(componentState.progress)}</span
+            >{fmt.format(componentState.easedProgress)}</span
           >
           &nbsp;
         </p>
diff --git a/src/components/HorizontalScroller/HorizontalScroller.mdx b/src/components/HorizontalScroller/HorizontalScroller.mdx
index 55b78b41..79c4b51d 100644
--- a/src/components/HorizontalScroller/HorizontalScroller.mdx
+++ b/src/components/HorizontalScroller/HorizontalScroller.mdx
@@ -18,7 +18,7 @@ To use the `HorizontalScroller` component, import it and provide the children co
 
 > 💡TIP: Use `lvh` or `svh` units instead of `vh` unit for the height, as [these units](https://www.w3.org/TR/css-values-4/#large-viewport-size) are more reliable on mobile or other devices where elements such as the address bar toggle between being shown and hidden.
 
-Use `showDebugInfo` prop to visualize the scroll progress and other useful debug information. The `Raw progress` indicates the vertical progress with values in the range 0...1 indicating the content being locked. The `Scroll Progress` value indicates the vertical progress mapped to clampStart and clampEnd values. By default these are 0 and 1 respectively. Finally, the `Progress` value indicates the horizontal scroll progress after applying stops and easing (if any).
+Use `showDebugInfo` prop to visualize the scroll progress and other useful debug information. The `Progress` indicates the vertical progress with values in the range 0...1 indicating the content being locked or a user-fed value to control the horizontal scroll position. The `Mapped Progress` value indicates the vertical progress mapped to mappedStart and mappedEnd values. By default these are 0 and 1 respectively. Finally, the `Eased Progress` value indicates the horizontal scroll progress after applying stops and easing (if any).
 
 [Demo](?path=/story/components-graphics-horizontalscroller--demo)
 
@@ -323,7 +323,7 @@ You can also integrate HorizontalScroller with `ScrollerBase` for a horizontal s
       width="fluid"
       height="100lvh"
       direction="right"
-      bind:scrollProgress={progress}
+      bind:progress
       scrubbed
       stops={[0.5]}
       handleScroll={false}
diff --git a/src/components/HorizontalScroller/HorizontalScroller.stories.svelte b/src/components/HorizontalScroller/HorizontalScroller.stories.svelte
index 1e58a827..893fccbc 100644
--- a/src/components/HorizontalScroller/HorizontalScroller.stories.svelte
+++ b/src/components/HorizontalScroller/HorizontalScroller.stories.svelte
@@ -49,8 +49,8 @@
   args={{
     children: DemoSnippet,
     height: '200lvh',
-    clampStart: -0.5,
-    clampEnd: 1.5,
+    mappedStart: -0.5,
+    mappedEnd: 1.5,
     showDebugInfo: true,
     scrubbed: true,
     stops: [0, 1],
diff --git a/src/components/HorizontalScroller/HorizontalScroller.svelte b/src/components/HorizontalScroller/HorizontalScroller.svelte
index 4664d5a8..10ae3098 100644
--- a/src/components/HorizontalScroller/HorizontalScroller.svelte
+++ b/src/components/HorizontalScroller/HorizontalScroller.svelte
@@ -13,7 +13,7 @@
     /** Height of the scroller container in CSS `vh` units. Set it to `100lvh` when using inside ScrollerBase. */
     height?: string;
     /** Bindable progress value. Ideal range: `[0-1]`. Bind ScrollerBase's progress to this prop. */
-    scrollProgress?: number;
+    progress?: number;
     /** Direction of movement*/
     direction?: 'left' | 'right';
     /** Content to scroll*/
@@ -31,9 +31,9 @@
     /** Whether to show debug info */
     showDebugInfo?: boolean;
     /** Modified starting scale. Default is 0 */
-    clampStart?: number;
+    mappedStart?: number;
     /** Modified ending scale. Default is 1 */
-    clampEnd?: number;
+    mappedEnd?: number;
   }
 
   let {
@@ -41,9 +41,9 @@
     class: cls = '',
     height = '200lvh',
     direction = 'right',
-    scrollProgress = $bindable(0),
-    clampStart = 0,
-    clampEnd = 1,
+    progress = $bindable(0),
+    mappedStart = 0,
+    mappedEnd = 1,
     children,
     stops = [],
     handleScroll = true,
@@ -54,22 +54,22 @@
   }: Props = $props();
 
   let componentState = $derived.by(() => ({
-    scrollProgress,
-    progress: progressTween.current,
+    progress,
+    mappedProgress,
+    easedProgress: easedProgress.current,
     direction,
-    clampStart,
-    clampEnd,
+    mappedStart,
+    mappedEnd,
     triggerStops: scrubbed ? stops : divisions,
     stops: stops,
     handleScroll,
     scrubbed,
     easing: ease,
     duration,
-    rawProgress,
   }));
 
-  let progressTween: Tween<number> = $state(
-    new Tween(clampStart, { duration, easing: ease })
+  let easedProgress: Tween<number> = $state(
+    new Tween(mappedStart, { duration, easing: ease })
   );
   let container: HTMLDivElement | undefined = $state(undefined);
   let containerHeight: number = $state(0);
@@ -78,19 +78,23 @@
   let contentWidth: number = $state(0);
   let screenHeight: number = $state(0);
   let divisions: number[] = $derived(
-    [...stops, clampStart, clampEnd].sort((a, b) => a - b)
+    [...stops, mappedStart, mappedEnd].sort((a, b) => a - b)
   );
   let divisionsCount: number = $derived.by(() => divisions.length - 1);
 
-  let rawProgress: number | 'user defined' = $state(0);
+  let mappedProgress: number = $state(0);
 
   // handles horizontal translation of the content
   let translateX: number = $derived.by(() => {
-    let processedProgress = clamp(progressTween.current, clampStart, clampEnd);
+    let processedProgress = clamp(
+      easedProgress.current,
+      mappedStart,
+      mappedEnd
+    );
     let normalisedProgress = processedProgress;
 
     normalisedProgress =
-      direction === 'right' ? processedProgress : clampEnd - processedProgress;
+      direction === 'right' ? processedProgress : mappedEnd - processedProgress;
 
     const translate = -(contentWidth - containerWidth) * normalisedProgress;
 
@@ -98,8 +102,8 @@
   });
 
   onMount(() => {
-    // Initialize scrollProgress to clampStart on mount
-    scrollProgress = clampStart;
+    // Initialize progress to mappedStart on mount
+    progress = mappedStart;
   });
 
   const scrollListener: Action = () => {
@@ -108,9 +112,9 @@
         passive: true,
       });
     } else {
-      // set rawProgress to user defined when handleScroll is false
-      rawProgress = 'user defined';
-      window.addEventListener('scroll', () => handleStops(scrollProgress), {
+      // set mappedProgress to user defined when handleScroll is false
+      //   mappedProgress = 'user defined';
+      window.addEventListener('scroll', () => handleStops(progress), {
         passive: true,
       });
     }
@@ -120,19 +124,19 @@
   function handleScrollFunction() {
     if (!container) return;
 
-    rawProgress =
+    progress =
       (-container?.offsetTop + window?.scrollY) /
       (containerHeight - screenHeight);
 
-    handleStops(rawProgress);
+    handleStops(progress);
   }
 
-  // updates progressTween based on stops and scrubbed settings
+  // updates easedProgress based on stops and scrubbed settings
   function handleStops(rawProgress: number) {
-    scrollProgress = map(rawProgress, 0, 1, clampStart, clampEnd);
+    mappedProgress = map(rawProgress, 0, 1, mappedStart, mappedEnd);
 
     if (!stops || stops.length === 0) {
-      progressTween.set(ease(map(rawProgress, 0, 1, clampStart, clampEnd)), {
+      easedProgress.set(ease(map(rawProgress, 0, 1, mappedStart, mappedEnd)), {
         duration: 0,
       });
       return;
@@ -141,8 +145,8 @@
     if (!scrubbed) {
       for (let i = 0; i < divisions.length; i++) {
         if (
-          scrollProgress > divisions[i] &&
-          scrollProgress <=
+          mappedProgress > divisions[i] &&
+          mappedProgress <=
             (divisions[i + 1] ?? divisions[divisions.length - 1])
         ) {
           const midPoint =
@@ -151,27 +155,27 @@
               divisions[i]) *
               0.5;
           if (
-            scrollProgress >= midPoint &&
-            progressTween.target !==
+            mappedProgress >= midPoint &&
+            easedProgress.target !==
               (divisions[i + 1] ?? divisions[divisions.length - 1])
           ) {
-            progressTween.set(
+            easedProgress.set(
               divisions[i + 1] ?? divisions[divisions.length - 1]
             );
             return;
           } else if (
-            scrollProgress < midPoint &&
-            progressTween.target !== divisions[i]
+            mappedProgress < midPoint &&
+            easedProgress.target !== divisions[i]
           ) {
-            progressTween.set(divisions[i]);
+            easedProgress.set(divisions[i]);
             return;
           }
         } else if (
-          scrollProgress <
-          divisions[0] + (divisions[1] ?? clampStart) * 0.5
+          mappedProgress <
+          divisions[0] + (divisions[1] ?? mappedStart) * 0.5
         ) {
-          if (progressTween.target !== divisions[0]) {
-            progressTween.set(divisions[0]);
+          if (easedProgress.target !== divisions[0]) {
+            easedProgress.set(divisions[0]);
             return;
           }
         } else {
@@ -182,19 +186,19 @@
       for (let i = 0; i < divisions.length; i++) {
         let oneByDivCount = 1 / divisionsCount;
 
-        let normalStart = i == 0 ? clampStart : oneByDivCount * i;
+        let normalStart = i == 0 ? mappedStart : oneByDivCount * i;
         let normalEnd =
-          i == divisionsCount - 1 ? clampEnd : oneByDivCount * (i + 1);
+          i == divisionsCount - 1 ? mappedEnd : oneByDivCount * (i + 1);
 
-        if (scrollProgress >= normalStart && scrollProgress < normalEnd) {
+        if (mappedProgress >= normalStart && mappedProgress < normalEnd) {
           let stopStart = divisions[i];
-          let stopEnd = divisions[i + 1] ?? clampEnd;
+          let stopEnd = divisions[i + 1] ?? mappedEnd;
           let newProgressVal =
             stopStart +
-            ease(map(scrollProgress, normalStart, normalEnd, 0, 1)) *
+            ease(map(mappedProgress, normalStart, normalEnd, 0, 1)) *
               (stopEnd - stopStart);
 
-          progressTween.set(newProgressVal, { duration: 0 });
+          easedProgress.set(newProgressVal, { duration: 0 });
           return;
         } else {
           continue;
diff --git a/src/components/HorizontalScroller/demo/DemoSnippet.svelte b/src/components/HorizontalScroller/demo/DemoSnippet.svelte
index 62f27205..128d145f 100644
--- a/src/components/HorizontalScroller/demo/DemoSnippet.svelte
+++ b/src/components/HorizontalScroller/demo/DemoSnippet.svelte
@@ -1,6 +1,6 @@
 <div style="width: 400vw; height: 100lvh;">
   <img
-    src="https://picsum.photos/1200/640?t=1"
+    src="https://picsum.photos/2400/1200?t=1"
     alt="Sample"
     style="width: 100%; height: 100%; object-fit: cover; padding: 0; margin: 0; background-color: #ccc;"
   />
diff --git a/src/components/HorizontalScroller/demo/withScrollerBase.svelte b/src/components/HorizontalScroller/demo/withScrollerBase.svelte
index 1b7c1d1f..2727d2df 100644
--- a/src/components/HorizontalScroller/demo/withScrollerBase.svelte
+++ b/src/components/HorizontalScroller/demo/withScrollerBase.svelte
@@ -35,7 +35,7 @@
     <HorizontalScroller
       height="100lvh"
       direction="right"
-      bind:scrollProgress={progress}
+      bind:progress
       scrubbed
       stops={[0.5]}
       handleScroll={false}