Skip to content

Nav3 migration experiments #1494

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

Closed
wants to merge 14 commits into from
Closed

Nav3 migration experiments #1494

wants to merge 14 commits into from

Conversation

ArnyminerZ
Copy link
Member

@ArnyminerZ ArnyminerZ commented May 26, 2025

Purpose

Tried experimenting a bit with the new Navigation 3 library. Unfortunately, right now we cannot even have something to test this out, since AppCompatActivity doesn't seem to be supported as of right now by Nav3.

Short description

Basically a lot has to change. We need:

  • Kotlin 2.1.21
  • KSP 2.0.1
  • API 36

Then, Activity Compose has to be upgraded to support Nav3 and AppCompatActivity. See issue on Google's Tracker and on the example repository for Nav3. Basically the main issue right now is that upon starting the app crashes with:

No NavigationEventDispatcher was provided via LocalNavigationEventDispatcherOwner

LocalNavigationEventDispatcherOwner doesn't even exist on the currently released version of Nav3. So we'd have to wait more.

Checklist

  • The PR has a proper title, description and label.
  • I have self-reviewed the PR.
  • I have added documentation to complex functions and functions that can be used by other modules.
  • I have added reasonable tests or consciously decided to not add tests.

Depends on android/nav3-recipes#6
Depends on https://issuetracker.google.com/issues/419208471

@ArnyminerZ ArnyminerZ self-assigned this May 26, 2025
@ArnyminerZ ArnyminerZ added the refactoring Internal improvement of existing functions label May 26, 2025
@ArnyminerZ
Copy link
Member Author

ArnyminerZ commented Jun 4, 2025

An update has been released for AppCompat, which should fix the issue: https://developer.android.com/jetpack/androidx/releases/appcompat#1.7.1

Edit: doesn't seem to work yet

# Conflicts:
#	app/src/main/kotlin/at/bitfire/davdroid/db/AppDatabase.kt
#	app/src/main/kotlin/at/bitfire/davdroid/ui/AccountsScreen.kt
Signed-off-by: Arnau Mora <[email protected]>
Signed-off-by: Arnau Mora <[email protected]>
Copy link

@rfc2822
Copy link
Member

rfc2822 commented Jul 8, 2025

As discussed, should be done for ICSx5 first

@rfc2822 rfc2822 closed this Jul 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactoring Internal improvement of existing functions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants