fix(ripple)!: rename focus methods to event handlers

PiperOrigin-RevId: 510549646
material-components · Feb 18, 2023 · 6e97717 · 6e97717
1 parent cde7ca0
commit 6e97717
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 8 deletions.
diff --git a/ripple/lib/ripple.ts b/ripple/lib/ripple.ts
@@ -63,11 +63,11 @@ export class Ripple extends LitElement {
     this.hovered = false;
   }
 
-  beginFocus() {
+  handleFocusin() {
     this.focused = true;
   }
 
-  endFocus() {
+  handleFocusout() {
     this.focused = false;
   }
 
@@ -102,7 +102,7 @@ export class Ripple extends LitElement {
   protected override update(changedProps: PropertyValues<this>) {
     if (changedProps.has('disabled') && this.disabled) {
       this.hovered = false;
-      this.endFocus();
+      this.focused = false;
       this.endPress();
     }
     super.update(changedProps);

diff --git a/ripple/lib/ripple_test.ts b/ripple/lib/ripple_test.ts
@@ -63,16 +63,16 @@ describe('ripple', () => {
       expect(surface).not.toHaveClass(RippleStateClasses.PRESSED);
     });
 
-    it('sets focused class on beginFocus()', async () => {
-      element.beginFocus();
+    it('sets focused class on handleFocusin()', async () => {
+      element.handleFocusin();
       await element.updateComplete;
 
       expect(surface).toHaveClass(RippleStateClasses.FOCUSED);
     });
 
-    it('removes focused class on endFocus()', async () => {
-      element.beginFocus();
-      element.endFocus();
+    it('removes focused class on handleFocusout()', async () => {
+      element.handleFocusin();
+      element.handleFocusout();
 
       expect(surface).not.toHaveClass(RippleStateClasses.FOCUSED);
     });