Skip to content

fix(material/list): remove pointer cursor for disabled selection list radios#33313

Open
EduardF1 wants to merge 2 commits into
angular:mainfrom
EduardF1:fix-disabled-selection-list-cursor
Open

fix(material/list): remove pointer cursor for disabled selection list radios#33313
EduardF1 wants to merge 2 commits into
angular:mainfrom
EduardF1:fix-disabled-selection-list-cursor

Conversation

@EduardF1
Copy link
Copy Markdown

Fixes #31937.

Summary

  • Ensures disabled single-selection list radios do not keep the MDC radio pointer cursor.
  • Adds a regression spec for disabled single-selection lists.

Validation

  • pnpm install --frozen-lockfile
  • pnpm exec prettier --check src/material/list/list.scss src/material/list/selection-list.spec.ts
  • pnpm exec stylelint src/material/list/list.scss
  • git --no-pager diff --check

Note: pnpm -s bazelisk test //src/material/list:unit_tests --test_output=errors was attempted after bazel mod deps --lockfile_mode=update, but Bazel analysis failed locally on Windows because no @@rules_sass+//src/toolchain:toolchain_type toolchain matched.

…on list radios

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@pullapprove pullapprove Bot requested review from andrewseguin and crisbeto May 28, 2026 23:00
@google-cla
Copy link
Copy Markdown

google-cla Bot commented May 28, 2026

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@EduardF1
Copy link
Copy Markdown
Author

@googlebot I signed it!

@EduardF1
Copy link
Copy Markdown
Author

Hi — I've signed the CLA but the check still shows as failing. Could someone re-trigger the CLA verification? Thanks!

@EduardF1
Copy link
Copy Markdown
Author

Hi! Could I request a review from @mmalerba and @wagnermaciel? You've both been very active reviewing Material list/selection-list changes. Thanks!

@EduardF1
Copy link
Copy Markdown
Author

I have signed the Google CLA (email: fischerszavaeduard@gmail.com). Could a maintainer please re-run the CLA check? Thank you!

ChangeDetectionStrategy.Eager does not exist in Angular's API.
Using Default (the implicit default) instead.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug(mat-selection-list): disabled selection list still shows cursor pointer on hover around radio

1 participant