Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(multiple): resolve host binding issues #30570

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

crisbeto
Copy link
Member

@crisbeto crisbeto commented Mar 2, 2025

Resolves the following issues with host bindings that weren't caught due to a lack of type checking:

  • The aria-describedby binding on mat-chip-listbox was doing nothing, because the _ariaDescribedBy property does not exist.
  • The focus handler on mat-chip-row was passing an event to _handleFocus even though one isn't required.
  • MatListOption was referencing a private method.
  • mat-sort-header was setting an invalid value to the _recentlyCleared signal.

Resolves the following issues with host bindings that weren't caught due to a lack of type checking:
* The `aria-describedby` binding on `mat-chip-listbox` was doing nothing, because the `_ariaDescribedBy` property does not exist.
* The `focus` handler on `mat-chip-row` was passing an event to `_handleFocus` even though one isn't required.
* `MatListOption` was referencing a private method.
* `mat-sort-header` was setting an invalid value to the `_recentlyCleared` signal.
@crisbeto crisbeto added the target: patch This PR is targeted for the next patch release label Mar 2, 2025
@crisbeto crisbeto requested a review from a team as a code owner March 2, 2025 08:49
@crisbeto crisbeto requested review from mmalerba and andrewseguin and removed request for a team March 2, 2025 08:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
target: patch This PR is targeted for the next patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant