fix(list): Add cursor: pointer for interactive list items (#4563)

Fixes #4557
material-components · Apr 3, 2019 · d2f0ccb · d2f0ccb
1 parent 98c0fee
commit d2f0ccb
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/packages/mdc-list/_mixins.scss b/packages/mdc-list/_mixins.scss
@@ -218,6 +218,13 @@
     }
   }
 
+  // Only change mouse cursor for interactive list items which are not disabled.
+  :not(.mdc-list--non-interactive) > :not(.mdc-list-item--disabled).mdc-list-item {
+    @include mdc-feature-targets($feat-structure) {
+      cursor: pointer;
+    }
+  }
+
   // Override anchor tag styles for the use-case of a list being used for navigation
   // stylelint-disable selector-max-type,selector-no-qualifying-type
   a.mdc-list-item {

diff --git a/packages/mdc-menu/_mixins.scss b/packages/mdc-menu/_mixins.scss
@@ -56,7 +56,6 @@
 
     .mdc-list-item {
       @include mdc-feature-targets($feat-structure) {
-        cursor: pointer;
         user-select: none;
       }
     }