Skip to content

[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

Merged
merged 7 commits into from
Dec 7, 2023

Conversation

MayuriKhinvasara
Copy link
Contributor

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
image

…lar android view)) and Image (Compose)) to display functionality and support of rendering an ultrahdr.
@MayuriKhinvasara MayuriKhinvasara added enhancement New feature or request new sample This denotes the creation of a new sample in the platform samples app labels Aug 30, 2023
@MayuriKhinvasara MayuriKhinvasara self-assigned this Aug 30, 2023
@MayuriKhinvasara MayuriKhinvasara changed the title [Sample] [Graphiocs] [Compose] UltraHDR image rendering in Compose. [Sample] [Graphics] [Compose] UltraHDR image rendering in Compose. Aug 30, 2023
Copy link
Contributor

@madebymozart madebymozart left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link

@Emilycroxall1996 Emilycroxall1996 left a comment

Choose a reason for hiding this comment

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

Na

…ctivity color mode controls for hdr and sdr (3) Remove fragment class and use the Sample annotation (4) Resolve minor readability comments
Copy link
Contributor

@yrezgui yrezgui left a 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

@MayuriKhinvasara
Copy link
Contributor Author

Make sure you're updating only the right sample dependencies

Updated, Ack

auto-merge was automatically disabled December 4, 2023 17:39

Pull request was closed

@madebymozart madebymozart requested a review from yrezgui December 5, 2023 16:52
@madebymozart
Copy link
Contributor

@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
@yrezgui yrezgui force-pushed the ultrahdr_compose_display_view branch from 10f0c80 to 11c18c4 Compare December 7, 2023 05:50
)

@Composable
fun RenderComposeView() {
Copy link
Contributor

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

Suggested change
fun RenderComposeView() {
fun RenderComposeScreen() {

@yrezgui yrezgui merged commit 0a58d34 into main Dec 7, 2023
@yrezgui yrezgui deleted the ultrahdr_compose_display_view branch December 7, 2023 07:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request new sample This denotes the creation of a new sample in the platform samples app
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants