-
Notifications
You must be signed in to change notification settings - Fork 344
[Sample] [Graphics] [Compose] UltraHDR image rendering in Compose. #89
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
Conversation
…lar android view)) and Image (Compose)) to display functionality and support of rendering an ultrahdr.
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
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.
Na
...dr/src/main/java/com/example/platform/graphics/ultrahdr/display/DisplayingUltraHDRCompose.kt
Outdated
Show resolved
Hide resolved
...dr/src/main/java/com/example/platform/graphics/ultrahdr/display/DisplayingUltraHDRCompose.kt
Outdated
Show resolved
Hide resolved
...dr/src/main/java/com/example/platform/graphics/ultrahdr/display/DisplayingUltraHDRCompose.kt
Outdated
Show resolved
Hide resolved
…to ultrahdr_compose_display_view
…ctivity color mode controls for hdr and sdr (3) Remove fragment class and use the Sample annotation (4) Resolve minor readability 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.
Make sure you're updating only the right sample dependencies
Updated, Ack |
@yrezgui Is there any more blockers on this one? |
* Update DisplayUltraHDRScreen logic to be Compose friendly * Add ImageView below to Compose Image composable Fetch colorMode details when screen is rendered * Update README.md
10f0c80
to
11c18c4
Compare
) | ||
|
||
@Composable | ||
fun RenderComposeView() { |
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.
Following convention across other Compose samples
fun RenderComposeView() { | |
fun RenderComposeScreen() { |
Description
This CL contains sample code for rendering UltraHDR Images in Compose (on API 34 and above).
How Has This Been Tested?
Test Configuration #1 - Pixel 7 Pro API 34 Beta (Android 14 Beta)
Checklist
I have performed a self-review of my code
I have commented my code, particularly in hard-to-understand areas
Screenshots
