-
Notifications
You must be signed in to change notification settings - Fork 10
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
[DST-462]Refactor <Dialog> component use Grid Areas #4164
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Skipped Deployment
|
🦋 Changeset detectedLatest commit: 201c1d4 The changes in this PR will be included in the next version bump. This PR includes changesets to release 9 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Coverage report
Show files with reduced coverage 🔻
Test suite run success573 tests passing in 78 suites. Report generated by 🧪jest coverage report action from 201c1d4 |
also documentation is missing |
Ah I also noticed, this change require a UI kit update right? @OsamaAbdellateef |
Yes you are right @sarahgm 👏🏻 |
Warning
This PR contains a breaking change! See below. This should be merged with #4187
Description
Our current API for the component is rather cumbersome and has some hard limitation when it comes to composability. We should improve this and align its API with the API of our other components (Components have their children “attached” e.g. Table.Cell)
What should be tested?
-The title and close button are vertically aligned
-The dialog is more composable
-The order of dialog sub-components always are the same
Reviewers:
@marigold-ui/developer