-
Notifications
You must be signed in to change notification settings - Fork 20
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
Enhancement/2176/options for reply component #2761
Conversation
✅ Deploy Preview for ons-design-system-preview ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
Looks good but maybe we should add the limit and description to one of the examples too
I did try this originally, couldn't think of a good descriptive label for a reply message.. Will have another go and add back in. |
What is the context of this PR?
#2176
How to review
Modify example, confirm new parameters are available inside
onsReply
. Tests pass.