feat(accordion): rename initialLoad state to hasRenderedOnce for clarity

scania-digital-design-system · Dec 18, 2024 · aa08be3 · aa08be3
1 parent f4bb00d
commit aa08be3
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/packages/core/src/components/accordion/accordion-item/accordion-item.tsx b/packages/core/src/components/accordion/accordion-item/accordion-item.tsx
@@ -36,7 +36,7 @@ export class TdsAccordionItem {
    * Private boolean to track if this is the initial load.
    * If true, we skip triggering the animation classes on first render.
    */
-  @State() private initialLoad = true;
+  @State() private hasRenderedOnce = false;
 
   /** Method for toggling the expanded state of the Accordion Item. */
   @Method()
@@ -64,7 +64,7 @@ export class TdsAccordionItem {
   render() {
     // If initialLoad is true, we do NOT attach the animation classes
     // so it won't animate on the very first render.
-    const shouldAnimate = this.animation !== 'none' && !this.initialLoad;
+    const shouldAnimate = this.animation !== 'none' && this.hasRenderedOnce;
 
     // Build up dynamic classes
     const classes = {
@@ -75,7 +75,7 @@ export class TdsAccordionItem {
       [`tds-accordion-item-animation-close-${this.animation}`]: shouldAnimate && !this.expanded,
     };
 
-    this.initialLoad = false;
+    this.hasRenderedOnce = true;
 
     return (
       <Host>