Skip to content

fix(cdk/drag-drop): use native popover to avoid stacking issues with preview - #28945

Merged
crisbeto merged 2 commits into
angular:mainfrom
crisbeto:popover-drag-drop-exploration
Apr 24, 2024
Merged

fix(cdk/drag-drop): use native popover to avoid stacking issues with preview#28945
crisbeto merged 2 commits into
angular:mainfrom
crisbeto:popover-drag-drop-exploration

Conversation

@crisbeto

Copy link
Copy Markdown
Member

Wraps the preview element in a native popover which allows it to always be rendered on top of everything and to avoid issues when the parent element has a transform.

Fixes#28889.

…lass
Moves the logic for creating and managing the preview into a separate class to make it a bit easier to manage. This new class is internal only.
…preview
Wraps the preview element in a native popover which allows it to always be rendered on top of everything and to avoid issues when the parent element has a `transform`.
Fixesangular#28889.
@crisbetocrisbeto added target: patch This PR is targeted for the next patch release merge: preserve commits When the PR is merged, a rebase and merge should be performed labels Apr 24, 2024
@crisbetocrisbeto added the action: merge The PR is ready for merge by the caretaker label Apr 24, 2024
@crisbetocrisbeto self-assigned this Apr 24, 2024
@crisbeto
crisbeto merged commit 7cd3f02 into angular:mainApr 24, 2024
crisbeto added a commit to crisbeto/material2 that referenced this pull request Apr 29, 2024
After angular#28945 the preview is inserted into a `popover` element which has a user agent styling of `color: canvastext`. These changes reset it to `inherit` to match the old behavior.
Fixesangular#28974.
@crisbetocrisbeto mentioned this pull request Apr 29, 2024
crisbeto added a commit to crisbeto/material2 that referenced this pull request Apr 29, 2024
After angular#28945 the preview is inserted into a `popover` element which has a user agent styling of `color: canvastext`. These changes reset it to `inherit` to match the old behavior.
Fixesangular#28974.
crisbeto added a commit to crisbeto/material2 that referenced this pull request Apr 30, 2024
After angular#28945 the preview is inserted into a `popover` element which has a user agent styling of `color: canvastext`. These changes reset it to `inherit` to match the old behavior.
Fixesangular#28974.
crisbeto added a commit that referenced this pull request May 1, 2024
After #28945 the preview is inserted into a `popover` element which has a user agent styling of `color: canvastext`. These changes reset it to `inherit` to match the old behavior.
Fixes#28974.
crisbeto added a commit that referenced this pull request May 1, 2024
After #28945 the preview is inserted into a `popover` element which has a user agent styling of `color: canvastext`. These changes reset it to `inherit` to match the old behavior.
Fixes#28974.
(cherry picked from commit a3b3211)
crisbeto added a commit that referenced this pull request May 1, 2024
After #28945 the preview is inserted into a `popover` element which has a user agent styling of `color: canvastext`. These changes reset it to `inherit` to match the old behavior.
Fixes#28974.
(cherry picked from commit a3b3211)
@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-botangular-automatic-lock-botBot locked and limited conversation to collaborators May 25, 2024
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: mergeThe PR is ready for merge by the caretakermerge: preserve commitsWhen the PR is merged, a rebase and merge should be performedtarget: patchThis PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug(CdkDragDrop): preview position is incorrect when inside of a popover using translate

2 participants

@crisbeto@zarend