-
Notifications
You must be signed in to change notification settings - Fork 213
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
chore: updated UI #1148
chore: updated UI #1148
Conversation
Reviewer's Guide by SourceryThis PR updates the UI of the app, primarily focusing on the navigation drawer and the home screen. The changes include visual enhancements, improved navigation, and a more consistent color scheme. Class diagram for BMDrawer widget changesclassDiagram
class BMDrawer {
+int selectedIndex
+createState() State
}
class _BMDrawerState {
-int currentIndex
+initState()
-updateSelectedIndex(int)
+build(BuildContext)
-_buildListTile(int, IconData?, String?, String, String, String?, String?)
}
BMDrawer <|-- StatefulWidget
_BMDrawerState <|-- State
BMDrawer ..> _BMDrawerState : creates
note for BMDrawer "Changed from StatelessWidget to StatefulWidget"
note for _BMDrawerState "New state class to manage drawer selection"
State diagram for navigation drawer selectionstateDiagram-v2
[*] --> Initial: Initialize with selectedIndex
Initial --> Selected: User taps menu item
Selected --> NavigateInternal: Internal route
Selected --> NavigateExternal: External link
Selected --> ShareContent: Share option
NavigateInternal --> UpdateUI: Update currentIndex
NavigateExternal --> OpenURL: Open external link
ShareContent --> ShowShareDialog: Show share dialog
UpdateUI --> Selected: Ready for next selection
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 @Jhalakupadhyay - I've reviewed your changes - here's some feedback:
Overall Comments:
- Please provide a more descriptive PR description explaining what changed and why, rather than just issue numbers. This helps reviewers understand the context and purpose of the changes.
Here's what I looked at during the review
- 🟢 General issues: all looks good
- 🟢 Security: all looks good
- 🟢 Testing: all looks good
- 🟢 Complexity: all looks good
- 🟢 Documentation: all looks good
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
ee86ffa
to
02cfb00
Compare
#1132
#1130
Summary by Sourcery
New Features: