-
Notifications
You must be signed in to change notification settings - Fork 66
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 support for Filled Button #274
Conversation
- Created a Mirai Filled Button Parser ✅ - Added example to the Mirai gallery ✅
home screen updated
home screen updated alignment
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.
Hey @iampranabray 👋🏻
Once again, Thank you for such amazing contributions.
I have some suggestions, please check my comments.
packages/mirai/lib/src/parsers/mirai_filled_button/mirai_filled_button_parser.dart
Outdated
Show resolved
Hide resolved
packages/mirai/lib/src/parsers/mirai_filled_button/mirai_filled_button_parser.dart
Outdated
Show resolved
Hide resolved
added comments are addressed++ - focusnode and materialStatesController removed - parseFilledButton added by removing parseElevated
freezed button file updated
@iampranabray Can you please resolve these conflicts? |
Yha sure |
Signed-off-by: Divyanshu Bhargava <divyanshub024@gmail.com>
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.
Thank you for this amazing contribution @iampranabray 🎉
Description
Related Issues
Closes #264
Type of Change