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

Support loading placeholders in compose previews #4982

Merged
merged 1 commit into from Feb 24, 2023

Conversation

sjudd
Copy link
Collaborator

@sjudd sjudd commented Dec 24, 2022

Composable placeholders already worked, but resource and drawable placeholders did not. Presumably some part of the production codepath isn't supported by the Android Studio renderer. The actual display of the resources seems to work fine, we just never get to that point in Glide's code. Debugging exactly what's happening seems mostly impossible. I don't see a way to view logs or debug code running in the Android Studio preview renderer.

For now we can work around whatever the issue is by rendering resources and drawables when in preview mode earlier than normal.

Composable placeholders already worked, but resource and drawable
placeholders did not. Presumably some part of the production codepath
isn't supported by the Android Studio renderer. The actual display of
the resources seems to work fine, we just never get to that point in
Glide's code. Debugging exactly what's happening seems mostly
impossible. I don't see a way to view logs or debug code running in the
Android Studio preview renderer.

For now we can work around whatever the issue is by rendering resources
and drawables when in preview mode earlier than normal.
@sjudd sjudd added the import-ready Indicates the PR is ready to be imported to Google. label Dec 24, 2022
@copybara-service copybara-service bot merged commit ef0e63d into bumptech:master Feb 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
import-ready Indicates the PR is ready to be imported to Google.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant