From 252b2cafe82aa8e08ef77e47107ffd89ab3e2ee5 Mon Sep 17 00:00:00 2001
From: MinamiFunakoshiTR <minami.funakoshi@thomsonreuters.com>
Date: Wed, 3 Dec 2025 14:59:52 -0500
Subject: [PATCH] clean up LottieForeground

---
 src/components/Lottie/Lottie.stories.svelte   |  4 +-
 src/components/Lottie/LottieForeground.svelte | 56 +++++++------------
 2 files changed, 23 insertions(+), 37 deletions(-)

diff --git a/src/components/Lottie/Lottie.stories.svelte b/src/components/Lottie/Lottie.stories.svelte
index a8a318a4..7790547a 100644
--- a/src/components/Lottie/Lottie.stories.svelte
+++ b/src/components/Lottie/Lottie.stories.svelte
@@ -83,7 +83,7 @@
 </Story>
 
 <Story name="With foregrounds">
-  <Lottie src={ForegroundSample} autoplay }>
+  <Lottie src={ForegroundSample} autoplay loop speed={1.5}>
     <LottieForeground
       startFrame={0}
       endFrame={50}
@@ -114,7 +114,7 @@
 
 <style lang="scss">
   :global {
-    .scroller-lottie-foreground {
+    .lottie-foreground-container {
       header {
         padding: 2rem;
         background-color: black;
diff --git a/src/components/Lottie/LottieForeground.svelte b/src/components/Lottie/LottieForeground.svelte
index e20e9377..ed74a21e 100644
--- a/src/components/Lottie/LottieForeground.svelte
+++ b/src/components/Lottie/LottieForeground.svelte
@@ -43,19 +43,16 @@
   });
 </script>
 
-<Block class={`scroller-lottie-foreground ${cls}`} {id}>
+<div class="lottie-foreground-container {cls}" {id}>
   {#if componentState?.currentFrame && componentState.currentFrame >= startFrame && componentState.currentFrame <= endFrame}
     <div
-      class="scroller-foreground"
+      class="lottie-foreground"
       in:fade={{ delay: 100, duration: 200 }}
       out:fade={{ delay: 0, duration: 100 }}
     >
       <!-- Text blurb foreground -->
       {#if text}
-        <Block
-          class="scroller-lottie-foreground-text {position.split(' ')[1]}"
-          {width}
-        >
+        <Block class="lottie-foreground-block {position.split(' ')[1]}" {width}>
           <div
             style="background-color: {backgroundColour};"
             class="foreground-text {position.split(' ')[0]}"
@@ -69,47 +66,34 @@
       {/if}
     </div>
   {/if}
-</Block>
+</div>
 
 <style lang="scss">
-  @use './../../scss/mixins' as mixins;
+  @use '../../scss/mixins' as mixins;
 
-  .scroller-foreground {
-    width: 100%;
-    height: 100%;
-  }
-
-  :global(.scroller-lottie-foreground) {
+  .lottie-foreground-container {
     position: absolute;
-    top: 50%;
-    left: 50%;
-    transform: translate(-50%, -50%);
-    width: 100%;
-    height: 100%;
+    inset: 0;
     display: flex;
     flex-direction: column;
     align-items: center;
     justify-content: center;
     z-index: 2;
-  }
 
-  .scroller-lottie-foreground-item {
-    width: 100%;
-    height: 100%;
-    position: absolute;
-    display: flex;
-    justify-content: center;
-    align-items: center;
+    .lottie-foreground {
+      width: 100%;
+      height: 100%;
+    }
   }
 
   :global {
-    .scroller-lottie-foreground-text {
+    .lottie-foreground-block {
       position: absolute;
       width: 100%;
       max-width: calc(mixins.$column-width-normal * 0.9);
       height: 100%;
 
-      @media (max-width: 1200px) {
+      &.center {
         left: 50%;
         transform: translateX(-50%);
       }
@@ -117,10 +101,12 @@
       &.left {
         left: 0;
       }
+
       &.right {
         right: 0;
       }
-      &.center {
+
+      @media (max-width: 1200px) {
         left: 50%;
         transform: translateX(-50%);
       }
@@ -128,28 +114,28 @@
 
     .foreground-text {
       position: absolute;
-      top: 50%;
       left: 50%;
-      transform: translate(-50%, -50%);
-      border-radius: 0.25rem;
-      background-color: white;
       width: 100%;
+      border-radius: 0.25rem;
       @include mixins.fpy-5;
       @include mixins.fpx-4;
       @include mixins.fm-0;
 
-      :global(*) {
+      * {
         margin: 0;
         padding: 0;
       }
 
       &.center {
         top: 50%;
+        transform: translate(-50%, -50%);
       }
+
       &.top {
         top: 0;
         transform: translate(-50%, 50%);
       }
+
       &.bottom {
         top: 100%;
         transform: translate(-50%, -150%);