Skip to content
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

Picklist: Added dragConfig to assign zIndex #13515

Merged
merged 2 commits into from
Aug 29, 2023

Conversation

SoyDiego
Copy link
Contributor

@SoyDiego SoyDiego commented Aug 19, 2023

Fix #13513

Added dragConfig to assign a zIndex value. More info here: https://material.angular.io/cdk/drag-drop/api#DragRefConfig
I added 1200 because dynamic modal and others always init in 1100, 1101, 1102.
If you consider another number, let me know and I will change.

Note: Also I formatted the file dynamicdialog.ts to pass the test.

CURRENT PROBLEM

problem picklist

AFTER SOLUTION

fixed picklist

@vercel
Copy link

vercel bot commented Aug 19, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
primeng-ssr-test ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 19, 2023 9:35am
1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
primeng ⬜️ Ignored (Inspect) Visit Preview Aug 19, 2023 9:35am

@SoyDiego SoyDiego mentioned this pull request Aug 19, 2023
@vercel vercel bot temporarily deployed to Preview – primeng-ssr-test August 19, 2023 09:35 Inactive
@SoyDiego SoyDiego changed the title Added config zIndex CDK Picklist: Added dragConfig to assign zIndex Aug 19, 2023
@gucal gucal self-requested a review August 26, 2023 00:03
@mertsincan mertsincan merged commit 294bdde into primefaces:master Aug 29, 2023
@mertsincan
Copy link
Member

Thanks a lot for your contribution!

Best Regards,

@SoyDiego SoyDiego deleted the 13513-fixed-picklist-zIndex branch August 31, 2023 19:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Component: Pick List
2 participants