Skip to content

fix(cdk/scrolling): make it easier to provide custom scrollable#33269

Merged
crisbeto merged 1 commit into
angular:mainfrom
crisbeto:scrollable-extend
May 18, 2026
Merged

fix(cdk/scrolling): make it easier to provide custom scrollable#33269
crisbeto merged 1 commit into
angular:mainfrom
crisbeto:scrollable-extend

Conversation

@crisbeto
Copy link
Copy Markdown
Member

Currently the ScrollDispatcher only accepts a CdkScrollable which makes it hard for users to register their own.

These changes make it so only the ScrollDispatcherTarget interface needs to be implemented instead.

@crisbeto crisbeto added the target: rc This PR is targeted for the next release-candidate label May 18, 2026
@pullapprove pullapprove Bot requested review from tjshiu and wagnermaciel May 18, 2026 10:48
@crisbeto crisbeto added the G This is is related to a Google internal issue label May 18, 2026
Currently the `ScrollDispatcher` only accepts a `CdkScrollable` which makes it hard for users to register their own.

These changes make it so only the `ScrollDispatcherTarget` interface needs to be implemented instead.
@crisbeto crisbeto force-pushed the scrollable-extend branch from df2c18f to d62646c Compare May 18, 2026 10:51
@crisbeto crisbeto added the action: merge The PR is ready for merge by the caretaker label May 18, 2026
@crisbeto crisbeto merged commit 332f786 into angular:main May 18, 2026
27 checks passed
@crisbeto
Copy link
Copy Markdown
Member Author

This PR was merged into the repository. The changes were merged into the following branches:

crisbeto added a commit that referenced this pull request May 18, 2026
Currently the `ScrollDispatcher` only accepts a `CdkScrollable` which makes it hard for users to register their own.

These changes make it so only the `ScrollDispatcherTarget` interface needs to be implemented instead.

(cherry picked from commit 332f786)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker area: cdk/scrolling G This is is related to a Google internal issue target: rc This PR is targeted for the next release-candidate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants