fix(signin button): signin button is clickable when token is autofill…

…ed by browser (#9610) * fix(signin button): signin button is clickable when token is autofilled by browser * fix: fix unit test & using input event instead of change event
kubernetes · Oct 31, 2024 · 1e49059 · 1e49059
1 parent 2f436f0
commit 1e49059
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/modules/web/src/login/component.spec.ts b/modules/web/src/login/component.spec.ts
@@ -153,7 +153,7 @@ describe('LoginComponent', () => {
       // set inputs and fire change events to trigger onChange()
       const token = fixture.debugElement.query(By.css(queries.token)).nativeElement;
       token.value = loginToken;
-      token.dispatchEvent(new Event('keyup'));
+      token.dispatchEvent(new Event('input'));
 
       submit();
 

diff --git a/modules/web/src/login/template.html b/modules/web/src/login/template.html
@@ -47,7 +47,7 @@
             placeholder="Bearer token"
             type="password"
             required
-            (keyup)="onChange($event)"
+            (input)="onChange($event)"
           />
         </mat-form-field>
         <mat-error