-
Notifications
You must be signed in to change notification settings - Fork 241
fix: Renaming Homescreen tabs #1428
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
base: development
Are you sure you want to change the base?
Conversation
Reviewer's GuideThis PR reorders the Homescreen tabs by swapping the Animation and Transition tabs and refactors their content: AnimationTab now hosts special icon-based animations while TransitionTab displays the basic animation options; comments, imports, and constructors are updated accordingly, and the pubspec.lock file is removed. Class diagram for updated AnimationTab and TransitionTabclassDiagram
class AnimationTab {
+State createState()
// Now displays special icon-based animations
}
class TransitionTab {
+State createState()
// Now displays basic animation options
}
AnimationTab --|> StatefulWidget
TransitionTab --|> StatefulWidget
class AniContainer {
+animation
+icon
+animationName
+index
}
AnimationTab o-- AniContainer
TransitionTab o-- AniContainer
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
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 there - I've reviewed your changes - here's some feedback:
- Consider refactoring the repeated AniContainer rows into a data-driven builder (e.g. iterate over a list of animation metadata) to reduce duplication and simplify future ordering changes.
- The import of constants.dart in transitiontab.dart isn’t used anymore—removing it will keep dependencies clean.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- Consider refactoring the repeated AniContainer rows into a data-driven builder (e.g. iterate over a list of animation metadata) to reduce duplication and simplify future ordering changes.
- The import of constants.dart in transitiontab.dart isn’t used anymore—removing it will keep dependencies clean.
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
Build StatusBuild successful. APKs to test: https://github.com/fossasia/badgemagic-app/actions/runs/17459619474/artifacts/3925447347. Screenshots |
hey @hpdang @mariobehling @AsCress need your reviews |
I don't have context on this change. If this is intentional, then looks good to me. |
Is there an issue linked as to why we need to make this change? Or any context? |
@adityastic yes, it was discussed in meeting as we have added new custom animations and the older animation is renamed as transition so that the correct sequence should be |
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.
Please follow best practices: https://blog.fossasia.org/open-source-developer-guide-and-best-practices-at-fossasia/
Add an issue to each PR. If there is no issue open an issue first before you make a PR.
Secondly add screenshots to the PR showing your changes.
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.
Please also resolve conflicts.
Co-authored-by: Mario Behling <[email protected]>
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.
Please address @sourcery-ai comments #1428 (review)
Fixes #1439
Changes
Changed the ordering of Homescreen tabs
Screenshots / Recordings
Checklist:
constants.dart
without hard coding any value.Summary by Sourcery
Reorder the Homescreen tabs and swap the content of the Animation and Transition tabs to better distinguish basic and special animations.
Enhancements:
Chores: