-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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
fix: dialog component drag behavior #13492
fix: dialog component drag behavior #13492
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Ignored Deployment
|
Hi @melloware. Could you please check out this bug please? I saw you have merged these changes in Prime React - primefaces/primereact#4760 |
I only have permissions to PrimeReact and PrimeFaces so all I can do is code review this PR @Algoritm211 |
@melloware , thank you very much, @cetincakiroglu, maybe you can take a look please? |
@cetincakiroglu, could you please merge it, before branch will change their state to outdated |
@SoyDiego , thank you very much for your approve, do you have permission to merge this PR? |
Hi @Algoritm211, unfortunately I dont have permission to merge :( |
Guys, @cetincakiroglu @gucal, sorry for disruption, but maybe any updates? |
Thanks a lot for your contribution! Best Regards, |
Defect Fixes
fix #13493
The main purpose of this PR - fix an issue with the drag of the dialog component when vertical or horizontal margins was applied to the
p-dialog
component. If the user adds a margin to the Dialog component will have unexpected behaviour during the dragging process. When the user tries to drag the popup to the left, it moves to the right.Current behaviour
first-1_vQ5eaKWM.mp4
Behavior after the fix
second-2_EjrfYrzf.mp4
Also in scope of this PR were solved problems with types assertion in code
This issue linked to the fixes for PrimeVue and PrimeReact