-
Notifications
You must be signed in to change notification settings - Fork 470
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
[Feature]: <Dialog /> component #830
Labels
Good first issue
Beginner-friendly issue for first-time contributors
Status: Help Wanted
We need help with this issue
Comments
christopherkindl
added
Status: Help Wanted
We need help with this issue
Good first issue
Beginner-friendly issue for first-time contributors
labels
Nov 24, 2023
I would like to work on this issue. Can you Please assign me this issue? |
sorry @Sanket-Arekar, somehow we did not see the comment. Now assigned! :) |
@Sanket-Arekar Just to conform, do you want to work on that? :) |
@christopherkindl I would like to work on it. |
@touha98 assigned! thank you :) |
🎉 This issue has been resolved in version 3.13.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
15 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Good first issue
Beginner-friendly issue for first-time contributors
Status: Help Wanted
We need help with this issue
What problem does this feature solve?
Introduce
<Dialog/>
component as shown in plain code usign Headless UI in Tremor Blocks (First two examples).Component should also use the transition used in those examples
Link to Headless UI documentation of
<Dialog>
What does the proposed API look like?
<Dialog/>
Props:open?: boolean
onClose?: (false) => void
static: boolean (default: false)
unmount: boolean (default: true)
<DialogPanel/>
Props:open?: boolean
Way of building:
The text was updated successfully, but these errors were encountered: