chore(maintenance): upgrade dependencies, migrate to Material 3, and resolve lint issues#922
Draft
dkhawk wants to merge 1 commit into
Draft
chore(maintenance): upgrade dependencies, migrate to Material 3, and resolve lint issues#922dkhawk wants to merge 1 commit into
dkhawk wants to merge 1 commit into
Conversation
…resolve lint issues - Upgraded outdated dependencies in gradle/libs.versions.toml: - compose-bom: 2026.03.00 -> 2026.05.01 - material: 1.13.0 -> 1.14.0 - kotlinxCoroutines: 1.10.2 -> 1.11.0 - screenshot & validation-api: 0.0.1-alpha14 -> 0.0.1-alpha15 - Refactored root build.gradle.kts to declare the Dokka plugin via its catalog alias (libs.plugins.dokka). - Enabled project-wide parallel builds (org.gradle.parallel=true) in gradle.properties. - Migrated 8 demo activities and fragments in maps-app from Material 2 to Material 3, resolving 24 UsingMaterialAndMaterial3Libraries lint warnings. - Fully qualified AnimatedVisibility call in MapsInLazyColumnActivity.kt to resolve ColumnScope resolution conflict inside the Material 3 Card. - Resolved ConfigurationScreenWidthHeight warning in MarkerClusteringActivity.kt by calculating screen dimensions in pixels using LocalWindowInfo containerSize instead of legacy LocalConfiguration screen Dp. - Resolved HardcodedText warning in activity_fragment_demo.xml by defining and using a string resource (swipe_to_switch).
Contributor
Code Coverage
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR performs essential maintenance by upgrading dependencies to their latest stable versions, completely migrating the demo application (
maps-app) to Jetpack Compose Material 3 (resolving 24 lint warnings), fixing other static analysis issues, and validating visual rendering.This PR is created in DRAFT state as we are waiting for the maps-ktx version update to finalize.
1. Dependency Upgrades 📦
Upgraded key dependencies in
gradle/libs.versions.toml:androidx.compose:compose-bom:2026.03.00➡️2026.05.01(stable Compose update)com.google.android.material:material:1.13.0➡️1.14.0(stable)org.jetbrains.kotlinx:kotlinx-coroutines-android&-test:1.10.2➡️1.11.0(stable)com.android.compose.screenshot&screenshot-validation-api:0.0.1-alpha14➡️0.0.1-alpha15(minor alpha update)build.gradle.ktsto declare the Dokka plugin via its catalog aliaslibs.plugins.dokkainstead of a hardcoded version string.gradle.properties.2. Material 3 Migration (
UsingMaterialAndMaterial3Libraries- 24 warnings resolved)Migrated 8 demo activities/fragments inside the
maps-appmodule from Material 2 (androidx.compose.material.*) to Material 3 (androidx.compose.material3.*) to ensure a unified, modern design system:CustomControlsActivity.kt,GoogleMapComposeFragment.kt,LocationTrackingActivity.kt,MapInColumnActivity.kt,MapsInLazyColumnActivity.kt,RecompositionActivity.kt,ScaleBarActivity.kt, andStreetViewActivity.kt.AnimatedVisibilitycall inMapsInLazyColumnActivity.ktto resolve aColumnScoperesolution conflict introduced by the Material 3Card's implicitColumnScopereceiver lambda.3. Other Lint Warnings Resolved 🛠️
ConfigurationScreenWidthHeight: Replaced the legacyLocalConfiguration.currentand screen DP property calculations with the modernLocalWindowInfo.current.containerSizeto pass exact pixel width/height dimensions directly toNonHierarchicalViewBasedAlgorithm.HardcodedText: Replaced a hardcoded string with a localized string resource (swipe_to_switch) inactivity_fragment_demo.xml.Verification & Visual Testing Status
validateDebugScreenshotTest). 100% of the screenshot tests passed successfully against baseline rendering!./gradlew testconcurrently via parallel builds, passing cleanly with zero errors.