Conversation
Summary of ChangesHello @rteyssandier, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request introduces a significant refactoring of the navigation system, specifically around the Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
There was a problem hiding this comment.
Code Review
This pull request reworks the navigation logic to adapt to an updated version of the nav3 library. The changes primarily involve refactoring MenuScene to compose a Scene object and updating MenuSceneStrategy to act as a scene decorator, which is a good approach. The dependency version for nav3Core is also updated accordingly. The changes are logical and align with the likely API changes in the new library version. I have one minor suggestion to improve code consistency and readability.
| val entry: NavEntry<FloconRoute>, | ||
| val menuContent: @Composable (expanded: Boolean) -> Unit, | ||
| val scene: Scene<FloconRoute>, | ||
| val menuContent: @Composable ((expanded: Boolean) -> Unit), |
There was a problem hiding this comment.
The extra parentheses around the function type for menuContent are redundant. For consistency with other function type declarations in this file, such as topBarContent on the next line and menuContent in the MenuSceneStrategy constructor, it's recommended to remove them to improve readability.
| val menuContent: @Composable ((expanded: Boolean) -> Unit), | |
| val menuContent: @Composable (expanded: Boolean) -> Unit, |
3baf2b5 to
d4bf0b2
Compare
c3e99e3 to
e064208
Compare
e064208 to
c8acf3d
Compare
Use https://issuetracker.google.com/issues/440333896