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

Add preview param support for @Paparazzi #1554

Open
wants to merge 37 commits into
base: akent/simple-preview/snapshots
Choose a base branch
from

Conversation

geoff-powell
Copy link
Collaborator

@geoff-powell geoff-powell commented Aug 24, 2024

Adds the ability to read @PreviewParameters for @composable's with the @paparazzi annotation

fixes #1687

@geoff-powell geoff-powell force-pushed the akent/simple-preview/plugin-implementation branch from b81c705 to f4a6041 Compare August 24, 2024 11:33
@geoff-powell geoff-powell force-pushed the akent/simple-preview/plugin-implementation branch from f4a6041 to 023b95f Compare August 24, 2024 11:36
@geoff-powell geoff-powell force-pushed the akent/simple-preview/plugin-implementation branch 2 times, most recently from 32a0b1d to e6c20a3 Compare August 26, 2024 10:05
@geoff-powell geoff-powell force-pushed the akent/simple-preview/plugin-implementation branch from e6c20a3 to 1fa7c72 Compare August 26, 2024 10:53
@geoff-powell geoff-powell force-pushed the akent/preview/preview-params branch 2 times, most recently from ae7db39 to 563618f Compare August 26, 2024 10:58
@geoff-powell geoff-powell force-pushed the akent/simple-preview/plugin-implementation branch 2 times, most recently from 7adf78e to 02b251d Compare August 26, 2024 15:30
@geoff-powell geoff-powell force-pushed the akent/preview/preview-params branch 2 times, most recently from ff32d9d to 1f4a052 Compare August 26, 2024 15:50
@geoff-powell geoff-powell force-pushed the akent/simple-preview/plugin-implementation branch from 02b251d to 8c72a58 Compare August 26, 2024 15:59
@geoff-powell geoff-powell force-pushed the akent/preview/preview-params branch 2 times, most recently from 32f231c to 6cf1056 Compare August 26, 2024 16:02
@geoff-powell geoff-powell force-pushed the akent/simple-preview/plugin-implementation branch from 8c72a58 to 280f576 Compare August 26, 2024 16:41
@geoff-powell geoff-powell force-pushed the akent/preview/preview-params branch 2 times, most recently from 3dd2b8f to aefdcf5 Compare August 26, 2024 17:23
@geoff-powell geoff-powell marked this pull request as ready for review August 26, 2024 17:23
@geoff-powell geoff-powell force-pushed the akent/preview/preview-params branch 3 times, most recently from 7dc395e to afd25fc Compare August 26, 2024 19:23
@geoff-powell geoff-powell force-pushed the akent/simple-preview/plugin-implementation branch from 280f576 to c76980f Compare August 27, 2024 14:14
@geoff-powell geoff-powell force-pushed the akent/simple-preview/plugin-implementation branch 2 times, most recently from 04f15e7 to 38b4698 Compare August 27, 2024 15:45
@geoff-powell geoff-powell force-pushed the akent/simple-preview/plugin-implementation branch from 38b4698 to a6c4f52 Compare August 27, 2024 16:01
@geoff-powell geoff-powell force-pushed the akent/preview/preview-params branch 3 times, most recently from 6375ea6 to 4de1774 Compare November 20, 2024 21:01
@@ -3,5 +3,6 @@ apply plugin: 'org.jetbrains.kotlin.plugin.compose'
apply plugin: 'com.vanniktech.maven.publish'

dependencies {
implementation libs.compose.runtime
compileOnly libs.compose.runtime
compileOnly libs.tools.layoutlib
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

nit: I don't love having to depend on this but we need access to android.content.res.Configuration used in preview. Maybe we could hardcode these values instead?

@geoff-powell geoff-powell force-pushed the akent/preview/preview-params branch 2 times, most recently from de71fe3 to 3f13d10 Compare November 21, 2024 14:20
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.

3 participants