-
Notifications
You must be signed in to change notification settings - Fork 4k
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 Picasso#rememberPainter extension function under new :picasso-compose module #2295
Conversation
cea2f21
to
9e69e9e
Compare
9e69e9e
to
780073d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice!
picasso-compose/gradle.properties
Outdated
@@ -0,0 +1,4 @@ | |||
POM_ARTIFACT_ID=picasso-compose | |||
POM_NAME=Picasso Compose | |||
POM_DESCRIPTION=A Compose UI plugin for Picasso. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
POM_DESCRIPTION=A Compose UI plugin for Picasso. | |
POM_DESCRIPTION=Compose UI support for Picasso. |
build.gradle
Outdated
@@ -28,6 +28,11 @@ buildscript { | |||
truth: 'com.google.truth:truth:1.1.3', | |||
robolectric: 'org.robolectric:robolectric:4.6.1', | |||
mockito: 'org.mockito:mockito-core:3.0.0', | |||
appCompat: 'androidx.appcompat:appcompat:1.3.1', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove?
build.gradle
Outdated
appCompat: 'androidx.appcompat:appcompat:1.3.1', | ||
drawablePainter: 'com.google.accompanist:accompanist-drawablepainter:0.19.0', | ||
composeUi: 'androidx.compose.ui:ui:1.1.1', | ||
composeUiTooling: 'androidx.compose.ui:ui-tooling:1.1.1', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove?
780073d
to
584f31f
Compare
584f31f
to
01cfad9
Compare
Thanks for providing this—this is great! Is there an issue we can follow to see when the |
Introduces a new :picasso-compose module which exposes a Picasso#rememberPainter function for use with Compose UI. To achieve this Picasso exposes a new .into(DrawableTarget) that way we can provide a PicassoDrawable to the Painter which will keep the API consistent with how we load .into(ImageView).
picasso-compose-ui.mp4