Overlay fixes and improvements #32227
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Currently the `CdkConnectedOverlay` directive has 23 inputs which can be quite verbose. These changes add a shorthand through the `cdkConnectedOverlay` attribute for the cases where these values aren't expected to change.
The connected overlay directive allows users to control its width using the `cdkConnectedOverlayMatchWidth` input. This can be tricky when it needs to match another element like the trigger. Since this is a common for cases like dropdowns, these changes add the `cdkConnectedOverlayMatchWidth` input to simplify it.
Explicitly hides the native backdrop from the CDK popover so it doesn't get rendered out.
crisbeto
requested review from
adolgachev and
andrewseguin
and removed request for
a team
crisbeto
added
the
merge: preserve commits
andrewseguin
approved these changes
Oct 31, 2025
|
Can the config be made injectable as well, so it inputs can be put into a provider? |
wagnermaciel
approved these changes
Oct 31, 2025
wagnermaciel
approved these changes
Oct 31, 2025
Adds the `CDK_CONNECTED_OVERLAY_DEFAULT_CONFIG` injection token to make it easier to configure the `CdkConnectedOverlay` directive.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Includes the following fixes and improvements to the CDK overlay:
fix(cdk/overlay): simplify public API of overlay directive
Currently the
CdkConnectedOverlaydirective has 23 inputs which can be quite verbose. These changes add a shorthand through thecdkConnectedOverlayattribute for the cases where these values aren't expected to change.fix(cdk/overlay): simplify matching the overlay to the trigger width
The connected overlay directive allows users to control its width using the
cdkConnectedOverlayMatchWidthinput. This can be tricky when it needs to match another element like the trigger.Since this is a common for cases like dropdowns, these changes add the
cdkConnectedOverlayMatchWidthinput to simplify it.fix(cdk/overlay): hide native backdrop
Explicitly hides the native backdrop from the CDK popover so it doesn't get rendered out.
fix(cdk/overlay): make it easier to set default for overlay directive
Adds the
CDK_CONNECTED_OVERLAY_DEFAULT_CONFIGinjection token to make it easier to configure theCdkConnectedOverlaydirective.