Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

bug(MatSelectionList): FocusKeyManager sets wrong parameter value for its skipPredicate() method #31775

Closed as not planned
Labels
needs triageThis issue needs to be triaged by the team
@AStrelkov

Description

Is this a regression?

  • Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was

No response

Description

In the following snippet of the MatSelectionList class, skipPredicate's parameter is wrong.

this._keyManager = new FocusKeyManager(this._items)
 .withHomeAndEnd()
 .withTypeAhead()
 .withWrap()
 .skipPredicate(() => this.disabled);

Reproduction

StackBlitz link:
Steps to reproduce:
1.
2.

Expected Behavior

Should be:

this._keyManager = new FocusKeyManager(this._items)
 .withHomeAndEnd()
 .withTypeAhead()
 .withWrap()
 .skipPredicate((item) => item.disabled);

Actual Behavior

Skipping doesn't work on the disabled options.

Environment

  • Angular: 19, 20
  • CDK/Material: 19, 20
  • Browser(s): Chrome
  • Operating System (e.g. Windows, macOS, Ubuntu): Windows 11

Metadata

Metadata

Assignees

No one assigned

    Labels

    needs triageThis issue needs to be triaged by the team

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions

        AltStyle によって変換されたページ (->オリジナル) /