fix(#1593): prevent onClick emission if step disabled

Infineon · Nov 19, 2024 · 2c2419b · 2c2419b
1 parent 693cd95
commit 2c2419b
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 4 deletions.
diff --git a/packages/components/src/components/stepper/step/step.tsx b/packages/components/src/components/stepper/step/step.tsx
@@ -52,18 +52,24 @@ export class Step {
             this.stepperState.setActiveStep(this.stepId, true)
         } 
     }
-
+    
     handleStepKeyDown(event: KeyboardEvent) {
         if (!this.disabled && this.stepperState.variant === 'default' && (this.clickable || this.complete) && event.key === 'Enter') {
             this.stepperState.setActiveStep(this.stepId, true)
         } 
     }
 
+    stopOnClickPropogation(event: Event) {
+        if (this.disabled) {
+            event.stopPropagation();
+        }
+    }
 
     render() {
         return (
             <div aria-current = {this.active ? 'step': false}
                 aria-disabled = {this.active || this.complete ? false : true}
+                onClick={ (e) => this.stopOnClickPropogation(e) }
                 class = {`step-wrapper ${this.stepId === 1 ? 'first-step': ''} 
                         ${this.error ? 'error': ''}
                         ${this.stepperState.variant}

diff --git a/packages/components/src/index.html b/packages/components/src/index.html
@@ -16,9 +16,8 @@
 
 </head>
 
-<body>  
-
+<body> 
+   
 </body>
 
-
 </html>