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
Removes requirement to render entire device Screen #497
Conversation
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.
LGTM
Since there were significant structural changes I've updated this PR based on the latest master branch. Thanks for your feedback, @liutikas, I've resolved your comments since the files have moved, but I did add the requested comments. |
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.
Looks good to me 👍
@jrodbx Anything preventing this from going in? |
It's on the queue for review, there's some other stuff I'd like to get in first + internal feature work to deliver. Hope to have a look soon. |
For what's worth androidx would love this feature to make 1.1 as well |
Apply cashapp/paparazzi#497 with some formatting tweaks to keep kt-lint happy. Test: N/A Bug: 253068501 Change-Id: Ieb1e3fa76fb4e1057e53a98fc6cbc81a848fba9f
paparazzi/paparazzi/src/test/java/app/cash/paparazzi/RenderingModeTest.kt
Outdated
Show resolved
Hide resolved
paparazzi/paparazzi/src/main/java/app/cash/paparazzi/Paparazzi.kt
Outdated
Show resolved
Hide resolved
I'll follow up with a PR to migrate the test as a plugin test, but this is great. thanks! |
Should this work with Compose? |
We use it with Compose views without issues. If you're having a specific failure perhaps open an issue? |
This adds support for
RenderMode.SHRINK
, allowing you to make view-only screenshots without saving an entire device frame.