-
Notifications
You must be signed in to change notification settings - Fork 0
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 reset button #18
Conversation
Coverage after merging issue_15 into main will be
Coverage Report for Changed Files
|
return InkWell( | ||
onTap: () async { | ||
avatarMakerController.restoreState(); | ||
if (onTap != null) onTap!(); |
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.
if (onTap != null) onTap!(); | |
onTap?.call(); |
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.
This is more of a suggestion, depending on wether we use the same lint rules as Kana to kanji the linter will ask for it
Coverage after merging issue_15 into main will be
Coverage Report for Changed Files
|
Co-authored-by: Xavier Chrétien <chretienxavier42@gmail.com>
Co-authored-by: Xavier Chrétien <chretienxavier42@gmail.com>
Coverage after merging issue_15 into main will be
Coverage Report for Changed Files
|
return InkWell( | ||
onTap: () async { | ||
avatarMakerController.restoreState(); | ||
if (onTap != null) onTap!(); |
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.
This is more of a suggestion, depending on wether we use the same lint rules as Kana to kanji the linter will ask for it
📖 Description
Add a reset button to make it easier for the users to reset the current customized avatar.
closes: #15
🖼️ Screenshots:
What the screenshot/video is presenting
2024-06-30.11-15-09.mp4
🧪 How to test the change?
☑️ Checklist before requesting a review
Please describe the checks you performed to verify your changes. Provide instructions so we can reproduce any specific checks to make sure things are working.
Tick the checkboxes to confirm if the checks were successful or not. (All that apply)
flutter run
at the example project and it works without any issues.flutter analyze
completes with "No issues found!"./CHANGELOG.md