Skip to content

Bump compose-navigation and add Material 3 sample#743

Open
Turtlepaw wants to merge 2 commits into
raamcosta:mainfrom
Turtlepaw:main
Open

Bump compose-navigation and add Material 3 sample#743
Turtlepaw wants to merge 2 commits into
raamcosta:mainfrom
Turtlepaw:main

Conversation

@Turtlepaw
Copy link
Copy Markdown

This PR fixes #739 by bumping compose-navigation. It also adds a copy of sample-wear that uses Wear Material 3.

studio64_qn15jEuFin.mp4

@raamcosta
Copy link
Copy Markdown
Owner

Do you think there's a reason to keep the other wear sample? I'm thinking I would just change that sample to use m3 rather than keeping both.

WDYT?

Thanks for this btw! 🙌

@Turtlepaw
Copy link
Copy Markdown
Author

Do you think there's a reason to keep the other wear sample? I'm thinking I would just change that sample to use m3 rather than keeping both.

WDYT?

Thanks for this btw! 🙌

Material 3 for Wear is still in alpha, and almost all Wear OS apps still use Wear Material (not m3), so I think it's worth keeping it at least until it's out of alpha.

@developer--
Copy link
Copy Markdown

Hi,
I'm getting NullPointerExceptions from Crashlytics for bottomsheets.
I opened ticket to google issue tracker, but could it be the reason, cause of this library doesn't uses latest navigation library versions internally?

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.

compose-destinations-wear doesn't support Wear Material 3

3 participants