Skip to content

Support Safe nav for AppList, AppDetail, Rule and RuleDetail screen#1170

Merged
lihenggui merged 33 commits intomainfrom
safe-nav
Mar 24, 2025
Merged

Support Safe nav for AppList, AppDetail, Rule and RuleDetail screen#1170
lihenggui merged 33 commits intomainfrom
safe-nav

Conversation

@juliewww
Copy link
Copy Markdown
Collaborator

@juliewww juliewww commented Mar 18, 2025


name: Pull request
about: Create a pull request
label: 'triage me'

Thank you for opening a Pull Request!
Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

  • Make sure to open a GitHub issue as a bug/feature request before writing your code! That way
    we can discuss the change, evaluate designs, and agree on the general idea
  • Ensure the tests and linter
    pass (./gradlew --init-script gradle/init.gradle.kts spotlessApply to automatically apply
    formatting)
  • Appropriate docs were updated (if necessary)

Is this your first Pull Request?

  • Run ./tools/setup.sh
  • Import the code formatting style as explained in the setup script.

Fixes #<issue_number_goes_here> 🦕

@juliewww juliewww changed the base branch from main to bottomSheet March 20, 2025 10:21
Base automatically changed from bottomSheet to main March 20, 2025 18:55
Comment thread .idea/codeStyles/Project.xml
@lihenggui lihenggui merged commit 266e7af into main Mar 24, 2025
4 checks passed
@lihenggui lihenggui deleted the safe-nav branch March 24, 2025 16:36
val unselectedIcon: ImageVector,
val iconTextId: Int,
val titleTextId: Int,
val route: KClass<*>,
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The TopLevelDestination looks could be treated as route type, no need to add an extra route.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants