-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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: Reflow while resizing does not reflow as expected #17243
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
/ok-to-test sha=72babc7 |
Unable to find test scripts. Please add necessary tests to the PR. |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/3165440446. |
UI Performance test run logs and artifacts: https://github.com/appsmithorg/appsmith/actions/runs/3165440446. Click to view performance test results
|
/ok-to-test sha=c9348c4 |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/3169177884. |
UI Performance test run logs and artifacts: https://github.com/appsmithorg/appsmith/actions/runs/3169177884. Click to view performance test results
|
dragDetails && | ||
!!dragDetails.draggedOn && | ||
dragDetails.draggedOn === widgetId) || | ||
isResizing) && |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is it possible to write a test case for this scenario @rahulramesha ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@marks0351, don't think we can, we will have to trigger reflow by dragging resize handle of the widget on cypress. That is similar to triggering reflow by dragging widget on canvas.
/ok-to-test sha=36d281e |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/3173643515. |
Description
getIsReflowEffectedSelector was not returning for when widget was resized but only on while draging. Added the additional Condition to fix it.
Fixes #17223
Type of change
How Has This Been Tested?
Manual UI
Checklist: