Skip to content

fix(cdk/overlay): scroll blocking sometimes clipping content #25204

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

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

Conversation

crisbeto
Copy link
Member

@crisbeto crisbeto commented Jul 1, 2022

Fixes that under some circumstances the cdk-global-scrollblock class was causing the body content to be clipped in the background.

Fixes #15051.

@crisbeto crisbeto added P3 An issue that is relevant to core functions, but does not impede progress. Important, but not urgent target: patch This PR is targeted for the next patch release labels Jul 1, 2022
@crisbeto crisbeto requested a review from jelbourn as a code owner July 1, 2022 07:01
Copy link
Member

@jelbourn jelbourn left a comment

Choose a reason for hiding this comment

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

lgtm

@jelbourn jelbourn added the action: merge The PR is ready for merge by the caretaker label Jul 11, 2022
@crisbeto crisbeto self-assigned this Jul 12, 2022
@crisbeto
Copy link
Member Author

This ended up causing some screenshot difference internally. Some were expected, but there were a handful that need more debugging.

@josephperrott josephperrott requested a review from a team as a code owner December 18, 2024 17:40
@josephperrott josephperrott requested review from amysorto and mmalerba and removed request for a team December 18, 2024 17:40
Fixes that under some circumstances the `cdk-global-scrollblock` class was causing the body content to be clipped in the background.

Fixes angular#15051.
@crisbeto crisbeto force-pushed the 15051/block-scroll-overflow branch from 1dff9dc to 093e778 Compare January 9, 2025 12:56
@mmalerba mmalerba removed their request for review February 20, 2025 00:16
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/overlay P3 An issue that is relevant to core functions, but does not impede progress. Important, but not urgent target: patch This PR is targeted for the next patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

block-scroll-strategy: some styles will cause not working as expected
3 participants