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
feat: ✨ add sd-dialog #668
Conversation
@Vahid1919 seems like all controls aren't working. Its not only for slots and the boolean for open and close the dialog. Please have a look on that! 👀 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Noticing some issues with the controls, let's look into that when you're back. Also what about the stories for Parts, Slots and Mouseless?
@coraliefeil I made a small change for the scrolling story that needs approval :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In general approval: Only some small questions + the question via chat, if a story makes sense to show how to prevent closign etc.
SolidElement.styles, | ||
css` | ||
:host { | ||
--width: 662px; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this coming from design?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes. I took the exact values from there.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
:host { | ||
--width: 335px; | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same: Coming from design?
Approved. |
Approved. THX |
# [@solid-design-system/components-v2.1.0](components/2.0.0...components/2.1.0) (2024-01-30) ### Features * ✨ add sd-dialog ([#668](#668)) ([753b6d8](753b6d8))
Description:
Added sd-dialog component. Closes #229.
Definition of Reviewable: