326b6174e9
## Description Current way of dropping building blocks already knows the widget id where it is going to be pasted. `getNewPositionsBasedOnMousePositions` did not take that into account and used to recalculate the canvas and container widgets. - This led to reflow and BBs getting reflowed further down(along with its container) This PR adds a fix to that ensuring no extra calculation and subsequently no further reflow. Fixes #`Issue Number` _or_ Fixes `Issue URL` > [!WARNING] > _If no issue exists, please create an issue first, and check with the maintainers if the issue is valid._ ## Automation /ok-to-test tags="@tag.Widget, @tag.Templates, @tag.IDE" ### 🔍 Cypress test results <!-- This is an auto-generated comment: Cypress test results --> > [!TIP] > 🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉 > Workflow run: <https://github.com/appsmithorg/appsmith/actions/runs/9379360946> > Commit: ba9c447fc662d2c1386b4caadd6dc04355c60082 > Cypress dashboard url: <a href="https://internal.appsmith.com/app/cypress-dashboard/rundetails-65890b3c81d7400d08fa9ee5?branch=master&workflowId=9379360946&attempt=1" target="_blank">Click here!</a> <!-- end of auto-generated comment: Cypress test results --> ## Communication Should the DevRel and Marketing teams inform users about this change? - [ ] Yes - [x] No <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit - **Bug Fixes** - Improved error handling when pasting building block widgets. - Enhanced logic for determining container widget when pasting into a specific widget. - **Tests** - Updated test descriptions for better readability. - Added parameter handling in test cases for widget pasting scenarios. <!-- end of auto-generated comment: release notes by coderabbit.ai --> |
||
|---|---|---|
| .. | ||
| client | ||
| server | ||
| util | ||