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

[VFX/SG] Fix Gradient Field which doesn't support HDR anymore #6333

Merged
merged 3 commits into from
Nov 22, 2021

Conversation

PaulDemeulenaere
Copy link
Contributor

@PaulDemeulenaere PaulDemeulenaere commented Nov 18, 2021


Purpose of this PR

There was a change of behavior introduced by UIToolkit with GradientField, the default isn't HDR anymore (since 2022.1.0a15): https://ono.unity3d.com/unity/unity/pull-request/132335/_/ui-toolkit/feature/property-drawers#comment-1028380

This PR restores the previous state allowing hdr value to be edited

Fix case 1381867


Testing status

Tested in VFX
_gradient_in_vfx

Tested in SG
_gradient_in_sg


Comments to reviewers

N/A

Copy link
Contributor

@julienamsellem julienamsellem left a comment

Choose a reason for hiding this comment

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

All good, thanks for the fix 👍

Copy link
Contributor

@VladNeykov VladNeykov left a comment

Choose a reason for hiding this comment

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

Auto-approving: small change, visuals provided showing the fix.
Thanks for resolving this! :)

Copy link
Contributor

@Nightmask3 Nightmask3 left a comment

Choose a reason for hiding this comment

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

LGTM!

@PaulDemeulenaere PaulDemeulenaere merged commit 2e176d3 into master Nov 22, 2021
@PaulDemeulenaere PaulDemeulenaere deleted the vfx/fix/1381867-gradient-not-hdr-anymore branch November 22, 2021 17:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants