netlify[bot]
commented
2 weeks ago Deploy Preview for tourmaline-bavarois-d21f10 ready!
| Name | Link |
|---|---|
| Latest commit | ff0f6e4e84309b9d6b9d146a3269e508661e6c77 |
| Latest deploy log | https://app.netlify.com/sites/tourmaline-bavarois-d21f10/deploys/672cc904f03f450008fc1105 |
| Deploy Preview | https://deploy-preview-1208--tourmaline-bavarois-d21f10.netlify.app |
| Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify site configuration.
vishanurag
swathivemula7
Added dragging visibility for all shapes
Related Issue
Closes: #1056
Description
This PR enhances the canvas drawing tool by enabling visible feedback during dragging actions. Previously, only finalized shapes and lines were visible after releasing the mouse, making it difficult for users to position and size shapes accurately while dragging. With this update, users can see shapes, lines elements in real-time as they draw.
Screenshots 📷
https://github.com/user-attachments/assets/0544018d-bffd-41c4-b31d-c0a9faa597d6
https://github.com/user-attachments/assets/3aa860ab-0942-4fce-a0db-f095be22ed93
@vishanurag @somyadipghosh Kindly review the PR at the earliest.. I have mentioned only Rectangle dragging visibilty in the issue #1056, but have implemented this feature for all the shapes in canvas editor..Consider upgrading the level..
Note: Some changes in the code are due to pulling the origin...
[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update
[x] My code follows the code style of this project.
[x] I have followed the contribution guidelines
[x] I have performed a self-review of my own code.
[x] I have ensured my changes don't generate any new warnings or errors.
[ ] I have updated the documentation (if necessary).
[x] I have resolved all merge conflicts.