fix(fab)!: remove disabled state

Fixes #4045 PiperOrigin-RevId: 517165006
material-components · Mar 16, 2023 · c368e7d · c368e7d
1 parent 9582e00
commit c368e7d
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 25 deletions.
diff --git a/fab/lib/_shared.scss b/fab/lib/_shared.scss
@@ -32,11 +32,6 @@
     vertical-align: top;
   }
 
-  :host([disabled]) {
-    cursor: default;
-    pointer-events: none;
-  }
-
   .md3-fab {
     display: inline-flex;
     border: none;

diff --git a/fab/lib/fab-shared.ts b/fab/lib/fab-shared.ts
@@ -24,8 +24,6 @@ export abstract class FabShared extends LitElement {
   static override shadowRootOptions:
       ShadowRootInit = {mode: 'open', delegatesFocus: true};
 
-  @property({type: Boolean}) disabled = false;
-
   @property() icon = '';
 
   @property() label = '';
@@ -52,7 +50,6 @@ export abstract class FabShared extends LitElement {
     return html`
       <button
         class="md3-fab md3-surface ${classMap(this.getRenderClasses())}"
-        ?disabled="${this.disabled}"
         aria-label="${ariaLabel}"
         @focus="${this.handleFocus}"
         @blur="${this.handleBlur}"
@@ -113,7 +110,6 @@ export abstract class FabShared extends LitElement {
   }
 
   private readonly renderRipple = () => {
-    return html`<md-ripple class="md3-fab__ripple" ?disabled="${
-        this.disabled}"></md-ripple>`;
+    return html`<md-ripple class="md3-fab__ripple"></md-ripple>`;
   };
 }
diff --git a/fab/md-fab_test.ts b/fab/md-fab_test.ts
@@ -46,7 +46,6 @@ describe('md-fab-extended', () => {
       const {harness} = await setupTest();
       expect(harness.element).toBeInstanceOf(MdFabExtended);
       expect(harness.element.lowered).toEqual(false);
-      expect(harness.element.disabled).toEqual(false);
       expect(harness.element.label).toEqual('');
       expect(harness.element.icon).toEqual('');
     });
@@ -62,20 +61,6 @@ describe('md-fab-extended', () => {
     });
   });
 
-  describe('disabled', () => {
-    it('get/set updates the disabled property on the native button element',
-       async () => {
-         const {harness, button} = await setupTest();
-         harness.element.disabled = true;
-         await env.waitForStability();
-
-         expect(button.disabled).toEqual(true);
-         harness.element.disabled = false;
-         await env.waitForStability();
-         expect(button.disabled).toEqual(false);
-       });
-  });
-
   describe('icon', () => {
     it('will generate an md-icon', async () => {
       const {harness, button} = await setupTest();