Skip to content

Add Grid sample#46

Open
JonEckenrode wants to merge 7 commits intomainfrom
add-grid-sample
Open

Add Grid sample#46
JonEckenrode wants to merge 7 commits intomainfrom
add-grid-sample

Conversation

@JonEckenrode
Copy link
Copy Markdown

@JonEckenrode JonEckenrode commented May 9, 2026

Add a Grid sample for the adaptive apps development hub samples page.

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request introduces a new Android project named Grid that demonstrates a three-column layout using the experimental Compose Grid API, including the complete project structure and build configuration. Feedback focuses on correcting several futuristic or unsupported versions for the Android SDK, Gradle Plugin, Kotlin, and Compose BOM to ensure build stability. Additionally, suggestions were made to improve code consistency by utilizing the version catalog for all dependencies, adopting idiomatic Compose functions like luminance(), and refining modifier logic.

Comment thread Grid/app/build.gradle.kts
Comment thread Grid/gradle/libs.versions.toml
Comment thread Grid/gradle/libs.versions.toml Outdated
Comment thread Grid/gradle/libs.versions.toml
Comment thread Grid/app/build.gradle.kts Outdated
Comment thread Grid/app/build.gradle.kts Outdated
Comment thread Grid/app/src/main/java/com/example/gridsample/MainActivity.kt Outdated
Comment thread Grid/app/src/main/java/com/example/gridsample/MainActivity.kt Outdated
Comment thread Grid/app/src/main/java/com/example/gridsample/MainActivity.kt Outdated
Comment thread Grid/gradle/libs.versions.toml Outdated
@JonEckenrode JonEckenrode changed the title Add Grid layout sample application and update .gitignore Add Grid sample May 10, 2026
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.

1 participant