Skip to content

fix(cdk-experimental/listbox): clean up the listbox API and make it work with forms - #24920

Merged
mmalerba merged 10 commits into
angular:mainfrom
mmalerba:cdk-listbox
May 26, 2022
Merged

mmalerba merged 10 commits into
angular:mainfrom
mmalerba:cdk-listbox

Conversation

@mmalerba

Copy link
Copy Markdown
Contributor

No description provided.

@mmalerba

Copy link
Copy Markdown
Contributor Author

Tests are a mess right now, but wanted to get some feeback on what I have so far. There's a couple questions I had while working on it which are called out in TODOs

Comment thread src/cdk-experimental/listbox/listbox.ts Resolved
Comment thread src/cdk-experimental/listbox/listbox.ts Resolved
Comment thread src/cdk-experimental/listbox/listbox.ts Resolved
Comment thread src/cdk-experimental/listbox/listbox.ts Resolved
Comment thread src/cdk-experimental/listbox/listbox.ts Resolved
Comment thread src/cdk-experimental/listbox/listbox.ts Resolved
Comment thread src/cdk-experimental/listbox/listbox.ts Resolved
Comment thread src/cdk-experimental/listbox/listbox.ts Resolved
Comment thread src/cdk-experimental/listbox/listbox.ts Resolved
Comment thread src/cdk-experimental/listbox/listbox.ts Resolved
Comment thread src/cdk-experimental/listbox/listbox.ts Resolved
Comment thread src/cdk-experimental/listbox/listbox.ts Resolved
Comment thread src/cdk-experimental/listbox/listbox.ts Resolved
Comment thread src/cdk-experimental/listbox/listbox.ts Resolved
Comment thread src/cdk-experimental/listbox/listbox.ts Resolved
Comment thread src/cdk-experimental/listbox/listbox.ts Resolved
Comment thread src/cdk-experimental/listbox/listbox.ts Resolved
Comment thread src/cdk-experimental/listbox/listbox.ts Resolved
Comment thread src/cdk-experimental/listbox/listbox.ts Resolved
Comment thread src/cdk-experimental/listbox/listbox.ts Resolved
Comment thread src/cdk-experimental/listbox/listbox.ts Resolved
Comment thread src/cdk-experimental/listbox/listbox.ts Resolved
Comment thread src/cdk-experimental/listbox/listbox.ts Resolved
Comment thread src/cdk-experimental/listbox/listbox.ts Resolved
@mmalerba
mmalerba force-pushed the cdk-listbox branch 2 times, most recently from 164460a to 81d3d90 Compare May 20, 2022 00:11
@mmalerba

Copy link
Copy Markdown
Contributor Author

PTAL, made some changes and left some responses above

Comment thread src/cdk-experimental/listbox/listbox.ts Resolved
Comment thread src/cdk-experimental/listbox/listbox.ts Resolved
Comment thread src/cdk-experimental/listbox/listbox.ts Resolved
Comment thread src/cdk-experimental/listbox/listbox.ts Resolved
Comment thread src/cdk-experimental/listbox/listbox.ts Resolved
@mmalerba
mmalerba marked this pull request as ready for review May 20, 2022 22:05
@mmalerba
mmalerba requested a review from a team May 20, 2022 22:05
@mmalerba
mmalerba requested a review from andrewseguin as a code owner May 20, 2022 22:05
@mmalerba mmalerba added the dev-app preview When applied, previews of the dev-app are deployed to Firebase label May 20, 2022
@github-actions

github-actions Bot commented May 20, 2022 •

Copy link
Copy Markdown

@devversion
devversion removed the request for review from a team May 23, 2022 13:23
@mmalerba

Copy link
Copy Markdown
Contributor Author

@jelbourn @crisbeto tests should hopefully be working now

@mmalerba mmalerba added the target: minor This PR is targeted for the next minor release label May 25, 2022

@crisbeto crisbeto left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One more nit otherwise LGTM

Comment thread src/cdk/collections/selection-model.ts Resolved
@mmalerba mmalerba assigned mmalerba and unassigned jelbourn May 25, 2022
@mmalerba mmalerba added the action: merge The PR is ready for merge by the caretaker label May 25, 2022
@mmalerba
mmalerba merged commit 0b59637 into angular:main May 26, 2022
@angular-automatic-lock-bot

Copy link
Copy Markdown

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot Bot locked and limited conversation to collaborators Jun 26, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker dev-app preview When applied, previews of the dev-app are deployed to Firebase target: minor This PR is targeted for the next minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants