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 custom component slots #750

Merged
merged 10 commits into from
Aug 16, 2022
Merged

Conversation

apedroferreira
Copy link
Member

@apedroferreira apedroferreira commented Aug 5, 2022

Fix placing components in custom components slot or slots props with any name.
Any custom component like this should have an acceptable behavior now, which should be a good baseline for the future even if this feature is still a bit experimental.

Screen.Recording.2022-08-10.at.18.31.52.mov

This fix involved a change to how the editor works: every component has at least one drop area corresponding to its rectangular shape, and another drop area for each empty slot inside it which takes priority.

Closes #713.

@render
Copy link

render bot commented Aug 5, 2022

@oliviertassinari oliviertassinari requested a deployment to fix-custom-component-slots - toolpad-db PR #750 August 5, 2022 17:49 — with Render Abandoned
@apedroferreira apedroferreira changed the base branch from master to drag-and-drop-fixes August 5, 2022 17:49
Base automatically changed from drag-and-drop-fixes to master August 8, 2022 15:40
@github-actions github-actions bot added the PR: out-of-date The pull request has merge conflicts and can't be merged label Aug 8, 2022
@github-actions github-actions bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged label Aug 8, 2022
@apedroferreira apedroferreira marked this pull request as ready for review August 9, 2022 21:34
@apedroferreira apedroferreira merged commit ff14689 into master Aug 16, 2022
@apedroferreira apedroferreira deleted the fix-custom-component-slots branch August 16, 2022 15:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Custom components with slots are broken
2 participants