Skip to content
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(examples): use Material 3 in example applications #2208

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

kkafar
Copy link
Member

@kkafar kkafar commented Jun 24, 2024

Description

Following up review discussion:

to PR

This PR introduces usage of Material 3 in our example applications, so that it is easier to test new Sheets & top app bar APIs.

The 1.9.0 version was chosen, because:

  1. This is the version I did all the testing on in feat!: iOS custom detents & Android form sheets #2045
  2. 1.10.0 introduces many dependency bumps, including appcompat & activity libs, which are known to be problematic with updates (we already had some issues in the past related to that)

Changes

  • Bump material to 1.9.0 in TestsExample
  • Bump material to 1.9.0 in Example
  • Bump material to 1.9.0 in FabricExample
  • Bump material to 1.9.0 in FabricTestExample

Test code and steps to reproduce

Application should work as normal & Material 3 components should be used.

Checklist

@kkafar kkafar force-pushed the @kkafar/use-material-3-in-examples branch from 7e1b582 to 4668c3c Compare June 24, 2024 11:26
Copy link
Member

@tboba tboba left a comment

Choose a reason for hiding this comment

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

LGTM! Just left a small comment there. It's great to introduce this change now, so that we could check if setting Material 3 version is safe in RN apps for sure 👍

@@ -132,6 +132,8 @@ dependencies {
} else {
implementation jscFlavor
}

implementation("com.google.android.material:material:1.9.0")
Copy link
Member

Choose a reason for hiding this comment

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

I'm worried if anyone that will try to install the Example app on Android will stumble upon the issue with too low version of Kotlin. Should we maybe add some comment here or create a README with prerequisites to install the app?

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.

2 participants