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

Fix resizing after drag & drop UI changes #1639

Merged
merged 10 commits into from
Feb 20, 2023
Merged

Conversation

apedroferreira
Copy link
Member

@apedroferreira apedroferreira commented Feb 7, 2023

Found a few issues with resizing after pushing the drag & drop UI changes, so here are some fixes/improvements:

  • Only allow resizing selected elements
  • Dashed hover outlines were confusing during resizing and dragging, so this hides them while resizing and dragging
  • Minimum grid row gap of 1 (feedback from Prakhar)

Preview: https://toolpad-pr-1639.onrender.com

@apedroferreira apedroferreira added the bug 🐛 Something doesn't work label Feb 7, 2023
@apedroferreira apedroferreira self-assigned this Feb 7, 2023
@oliviertassinari oliviertassinari requested a deployment to resizing-fixes - toolpad-db PR #1639 February 7, 2023 18:29 — with Render Abandoned
@apedroferreira apedroferreira changed the title Resizing fixes Fix resizing after drag & drop UI changes Feb 7, 2023
@oliviertassinari oliviertassinari temporarily deployed to resizing-fixes - toolpad PR #1639 February 7, 2023 18:30 — with Render Destroyed
@apedroferreira apedroferreira marked this pull request as ready for review February 7, 2023 18:31
@apedroferreira
Copy link
Member Author

Just noticed there are console errors now too... will include the fix here.

@Janpot
Copy link
Member

Janpot commented Feb 8, 2023

so it wasn't possible to resize from inside an unselected element

For what it's worth, I don't think we should support resizing from non-selected components.

@apedroferreira
Copy link
Member Author

For what it's worth, I don't think we should support resizing from non-selected components.

Yep I was just seeing if someone agreed on that, will make it only possible to resize the selected component.

@oliviertassinari oliviertassinari temporarily deployed to resizing-fixes - toolpad PR #1639 February 8, 2023 16:26 — with Render Destroyed
@oliviertassinari oliviertassinari temporarily deployed to resizing-fixes - toolpad PR #1639 February 8, 2023 16:29 — with Render Destroyed
@oliviertassinari oliviertassinari temporarily deployed to resizing-fixes - toolpad PR #1639 February 8, 2023 19:08 — with Render Destroyed
@apedroferreira apedroferreira requested a review from a team February 20, 2023 15:18
@oliviertassinari oliviertassinari requested a deployment to resizing-fixes - toolpad-db PR #1639 February 20, 2023 15:18 — with Render Abandoned
@oliviertassinari oliviertassinari temporarily deployed to resizing-fixes - toolpad PR #1639 February 20, 2023 15:20 — with Render Destroyed
@apedroferreira apedroferreira merged commit 2396c66 into master Feb 20, 2023
@apedroferreira apedroferreira deleted the resizing-fixes branch February 20, 2023 17:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 Something doesn't work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants